👨‍💻 Simplify the workflow for running ty

[burgholzer]

Description

Another small improvement for how ty is being run based on the experience gained when rolling this support out across repositories.

Checklist:

• The pull request only contains commits that are focused and relevant to this change.
• I have added appropriate tests that cover the new/changed functionality.
• I have updated the documentation to reflect these changes.
• I have added entries to the changelog for any noteworthy additions, changes, fixes, or removals.
• I have added migration instructions to the upgrade guide (if needed).
• The changes follow the project's style guidelines and introduce no new warnings.
• The changes are fully tested and pass the CI checks.
• I have reviewed my own code changes.

[denialhaag]

LGTM!

[burgholzer]

@coderabbitai full review

[coderabbitai]

✅ Actions performed

Full review triggered.

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

• Chores
  • Streamlined CI/CD workflow for type-checking processes
• Documentation
  • Updated changelog documenting workflow improvements

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

Remove redundant UV setup options and simplify the ty type-check step in the GitHub Actions Python linter workflow; add an Unreleased changelog entry documenting this change.

Changes

Cohort / File(s)	Summary
Workflow simplification /.github/workflows/reusable-python-linter.yml	Removed python-version and activate-environment from the UV setup step; replaced the multi-command shell block for running ty with a single uv run --no-sync ty check invocation.
Changelog update CHANGELOG.md	Added an "Unreleased" entry describing the workflow simplification and added a PR reference link.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Focus review on the modified workflow step for correctness of the new uv command and any required flags.
• Verify changelog entry formatting and PR link.

Possibly related PRs

• 🔧 Use locked version of ty for linter workflow to ensure stability #255 — Modifies the same CI workflow's ty step and UV setup; closely related to the workflow changes here.
• 🐍🚨 Modernize and fix Python linter workflow #254 — Another PR that altered the linter workflow's ty invocation; relevant for context and consistency.
• 🚨 Enable Astral's type checker ty core#1333 — Changes CI handling around enabling ty; related to how ty is integrated in workflows.

Poem

🐰
I nibbled through the workflow tree,
Trimmed extra hops so ty runs free.
One swift command, the path made light,
A cleaner hop — and all feels right. ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: simplifying the workflow for running the ty tool, which aligns with the primary modifications shown in the changeset.
Description check	✅ Passed	The description provides context about the improvement, confirms all checklist items are completed, and aligns with the template structure, though it lacks specific issue references or detailed technical context.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch simplify-running-ty

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 93af7bb and c1a2a6e.

📒 Files selected for processing (1)

• .github/workflows/reusable-python-linter.yml (1 hunks)

🔇 Additional comments (1)

.github/workflows/reusable-python-linter.yml (1)

58-62: The implementation correctly uses the two-step approach to balance type-checking accuracy with build speed.

The ty check step properly combines uv sync --no-install-project (which installs all dependencies including dev groups) with uv run --no-sync ty check (which runs ty without re-syncing). This ensures ty has access to project dependencies for accurate type checking while avoiding the overhead of installing the project itself.

Both flags are well-established:

• --no-install-project available since uv 0.3.3
• --no-sync available in uv 0.4.x and later

Since setup-uv v7.1.4 installs the latest uv by default, compatibility is not a concern. The implementation is sound and ready.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 20b91b0 and 93af7bb.

📒 Files selected for processing (2)

• .github/workflows/reusable-python-linter.yml (1 hunks)
• CHANGELOG.md (2 hunks)

🔇 Additional comments (1)

CHANGELOG.md (1)

12-14: LGTM! Changelog entry and PR link are properly formatted.

The "Changed" entry accurately reflects the workflow simplification and follows the established format. The PR link is correctly positioned in the PR links section.

Also applies to: 202-202