Skip to content

Update template#15

Draft
NKeleher wants to merge 4 commits into
mainfrom
update-template
Draft

Update template#15
NKeleher wants to merge 4 commits into
mainfrom
update-template

Conversation

@NKeleher

Copy link
Copy Markdown
Contributor

Pull Request Summary 🚀

What does this PR do? 📝

Why is this change needed? 🤔

How was this implemented? 🛠️

How to test or reproduce ? 🧪

Screenshots (if applicable) 📷

Checklist ✅

  • I have run and tested my changes locally
  • I have limited this PR to less than 1000 lines of code change (if not, explain why)
  • I have updated/added tests to cover my changes (if applicable)
  • I have updated/added requirements to cover my changes (if applicable)
  • I have run linting and formatting on any code changes (if applicable)
  • I have updated the documentation (README, etc.) accordingly

Reviewer Emoji Legend

:code: Meaning
😃👍💯 :smiley: :+1: :100: I like this...

...and I want the author to know it! This is a way to highlight positive parts of a code review.
⭐⭐⭐ :star: :star: :star: Important to fix before PR can be approved...

And I am providing reasons why it needs to be addressed as well as suggested improvements.
⭐⭐ :star: :star: Important to fix but non-blocking for PR approval...

And I am providing suggestions where it could be improved either in this PR or later.
:star: Give this some thought but non-blocking for PR approval...

...and consider this a suggestion, not a requirement.
:question: I have a question.

This should be a fully formed question with sufficient information and context that requires a response.
📝 :memo: This is an explanatory note, fun fact, or relevant commentary that does not require any action.
:pick: This is a nitpick.

This does not require any changes and is often better left unsaid. This may include stylistic, formatting, or organization suggestions and should likely be prevented/enforced by linting if they really matter
♻️ :recycle: Suggestion for refactoring.

Should include enough context to be actionable and not be considered a nitpick.

NKeleher and others added 4 commits July 20, 2026 13:02
…e QoL

- Swap markdownlint/mdformat for panache (panache-cli dev dep, panache.toml,
  jolars/panache-pre-commit hooks, jolars.panache VS Code extension)
- Add src/project_name package with uv_build build backend
- Add tests/ with pytest + pytest-cov; run tests in CI
- Move dev deps to PEP 735 [dependency-groups]; declare sqlfluff
- Rework Justfile: grouped recipes, default list, test/check-all recipes,
  fix broken activate-venv (uv shell) and Windows clean
- Add minimal Quarto scaffold in docs/
- Rename pre-commit workflow to ci.yml with pre-commit + tests jobs

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant