Skip to content

build: copier update from template 0.29.1#176

Merged
ichoosetoaccept merged 1 commit intomainfrom
02-17-build-copier-update-from-template-0.29.1
Feb 18, 2026
Merged

build: copier update from template 0.29.1#176
ichoosetoaccept merged 1 commit intomainfrom
02-17-build-copier-update-from-template-0.29.1

Conversation

@ichoosetoaccept
Copy link
Member

@ichoosetoaccept ichoosetoaccept commented Feb 17, 2026

Description

Syncs project tooling with copier template v0.29.1. Key changes:

  • Fix .copier-answers.yml: Set cli_framework: none (was typer, but cyclopts comes transitively from fastmcp)
  • Remove spurious typer dep from pyproject.toml
  • Add ci-pass aggregation job in GitHub Actions for cleaner status checks
  • Enhanced ruff config: clearer rule comments, added FURB/FLY/PGH rules, S404 in test ignores, TRY003 globally ignored
  • Improved pre-commit hooks: pytest -q flag, verbose: true for better output
  • Template update script: writes to /dev/tty when available to bypass hook runner capture
  • Remove unused # noqa: S404 from tests/test_gh.py (now covered by per-file-ignores)

Checklist

  • Tests added/updated
  • Documentation updated (if applicable)
  • poe check passes
  • Commit messages follow conventional commits

Copy link
Member Author

ichoosetoaccept commented Feb 17, 2026

@greptile-apps
Copy link

greptile-apps bot commented Feb 17, 2026

Greptile Summary

Updates copier template from 0.27.11 to 0.29.1, bringing project tooling and configuration in sync with latest template standards. Key improvements include a new CI status check job, streamlined documentation, enhanced ruff linting rules with better organization, and improved git hook output handling.

Major changes:

  • Fixed .copier-answers.yml to correctly specify cli_framework: none (resolves previous thread concern about typer dependency)
  • Added ci-pass aggregation job in GitHub Actions for cleaner status checks
  • Enhanced ruff configuration with clearer rule comments and added TRY003 to ignore list
  • Updated pytest hooks with -q flag and verbose: true for better output control
  • Improved template update script to write to /dev/tty when available to bypass hook runner capture
  • Updated poe update-template command to include --defaults flag and uv sync --upgrade
  • Minor dependency updates: backrefs, cyclopts, py-key-value-aio, uvicorn

Confidence Score: 4/5

  • This PR is safe to merge with low risk - it's a routine template update with well-understood changes
  • Score reflects that this is a standard copier template update that successfully addresses the previous review concern about cli_framework. The changes are configuration and documentation improvements with no breaking changes. Minor point deducted due to the --defaults flag in update-template command which could potentially reset custom copier answers on future updates, though this appears intentional.
  • Pay attention to pyproject.toml - verify the --defaults flag behavior in the update-template command doesn't unintentionally reset custom copier configuration on future updates

Important Files Changed

Filename Overview
.copier-answers.yml Updated template version to 0.29.1 and correctly set cli_framework: none to match actual cyclopts usage
.github/workflows/ci.yml Added ci-pass job for better status checks using alls-green action with proper job dependencies
prek.toml Added -q flag to pytest hooks and verbose: true for better output control in pre-commit hooks
pyproject.toml Enhanced ruff rules with better comments, added TRY003 to ignore list, improved per-file ignores, and updated update-template command with --defaults flag

Flowchart

flowchart TD
    A[CI Trigger] --> B[links job]
    A --> C[quality job]
    A --> D[tests job]
    
    B --> E[ci-pass job]
    C --> E
    D --> E
    
    E --> F{Check all jobs<br/>using alls-green}
    F -->|All passed or<br/>allowed-skips| G[✓ CI Pass]
    F -->|Any required<br/>job failed| H[✗ CI Fail]
    
    style E fill:#e1f5e1
    style G fill:#90EE90
    style H fill:#FFB6C6
Loading

Last reviewed commit: 44747e2

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

8 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@ichoosetoaccept ichoosetoaccept force-pushed the 02-17-build-copier-update-from-template-0.29.1 branch from 03f4e4a to f79a621 Compare February 17, 2026 23:32
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

8 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@ichoosetoaccept ichoosetoaccept force-pushed the 02-17-build-copier-update-from-template-0.29.1 branch from f79a621 to 44747e2 Compare February 17, 2026 23:52
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

9 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Copy link
Member Author

ichoosetoaccept commented Feb 18, 2026

Merge activity

  • Feb 18, 7:17 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Feb 18, 7:17 AM UTC: @ichoosetoaccept merged this pull request with Graphite.

@ichoosetoaccept ichoosetoaccept merged commit 3f2a4f7 into main Feb 18, 2026
12 checks passed
@ichoosetoaccept ichoosetoaccept deleted the 02-17-build-copier-update-from-template-0.29.1 branch February 18, 2026 07:17
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