Skip to content

chore: sync template from tschm/.config-templates@77-action-with-explicit-devcontainer-copy#195

Merged
tschm merged 1 commit intomainfrom
sync/update
Sep 24, 2025
Merged

chore: sync template from tschm/.config-templates@77-action-with-explicit-devcontainer-copy#195
tschm merged 1 commit intomainfrom
sync/update

Conversation

@tschm
Copy link
Copy Markdown
Member

@tschm tschm commented Sep 24, 2025

This PR updates configuration files from
tschm/.config-templates@77-action-with-explicit-devcontainer-copy.

Summary by CodeRabbit

  • Chores

    • Streamlined ignore rules for a cleaner repository setup.
  • Build

    • Quieter build, test, and documentation output for easier reading.
    • Clearer error reporting when builds fail.
  • Developer Experience

    • Ensures user-local binaries are available on the command line by default.
    • Smarter detection of the source folder with a sensible default.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Sep 24, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Updates ignore patterns in .gitignore, adds silent flags to Makefile targets, prepends a local bin path via Taskfile.yml env, hardens build failure handling in taskfiles/build.yml, and revises source folder detection logic in taskfiles/quality.yml.

Changes

Cohort / File(s) Summary of Changes
Git ignore updates
\.gitignore
Added header for Python template; ignored _tests, .pdm.toml, .env; adjusted Python bytecode patterns to *.py[cod]; clarified __pycache__/; removed several tool-specific blocks; reorganized sections.
Makefile verbosity adjustments
Makefile
Added -s (silent) to commands in install, clean, test, book (and subtargets), and check to reduce command echo/output.
Task runner env path
Taskfile.yml
Introduced PATH env: {{.HOME}}/.local/bin:{{.PATH}} to prepend user-local bin directory.
Task automation: build
taskfiles/build.yml
Appended guarded exit to uvx hatch build: prints error and exits with code 1 on failure; still conditional on presence of pyproject.toml.
Task automation: quality
taskfiles/quality.yml
Replaced find-based source dir discovery with fd logic; defaults SOURCE_FOLDER to src if unset; retained else-branch for missing pyproject.toml; kept legacy approach as commented reference.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Dev as Developer
  participant Task as Task Runner
  participant Build as Build Task (uvx hatch)
  participant QC as Quality Task (deptry)

  Note over Task,Build: Build flow with guarded failure
  Dev->>Task: run build
  Task->>Build: uvx hatch build (when pyproject.toml exists)
  alt build succeeds
    Build-->>Task: success (0)
  else build fails
    Build-->>Task: non-zero
    Task->>Task: echo "Build failed" and exit 1
  end

  Note over Task,QC: Quality flow with source detection
  Dev->>Task: run quality
  Task->>QC: determine SOURCE_FOLDER via fd (default "src" if unset)
  QC-->>Dev: run deptry on detected folder (or handle no pyproject.toml)
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

I twitch my ears at quieter make,
Paths hop ahead for tools to take.
If builds misstep, we fail with grace,
And sniff out src with careful pace.
In burrows clean, my whiskers preen—
Ignore the noise; keep changes lean. 🐇✨

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sync/update

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a7706cf and 9288dee.

📒 Files selected for processing (5)
  • .gitignore (4 hunks)
  • Makefile (1 hunks)
  • Taskfile.yml (1 hunks)
  • taskfiles/build.yml (1 hunks)
  • taskfiles/quality.yml (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tschm tschm merged commit 3330f40 into main Sep 24, 2025
8 of 9 checks passed
@tschm tschm deleted the sync/update branch September 24, 2025 18:51
@coderabbitai coderabbitai Bot mentioned this pull request Dec 29, 2025
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