Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis PR updates GitHub Actions workflow files to use newer major versions of core actions (checkout, setup-python, cache, CodeQL, and github-script) across documentation, CI, linting, CodeQL analysis, coverage, and publish workflows, without changing workflow logic. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue, and left some high level feedback:
- Several upgraded actions (e.g.,
actions/checkout@v6,actions/setup-python@v6,actions/cache@v5,github/codeql-action@v4,actions/github-script@v9) don’t currently exist as published major versions; please confirm these tags are valid and adjust to the latest actually released major versions. - For workflow stability, consider upgrading incrementally to the latest known major versions (e.g., from v3 to v4) and verifying each workflow run, rather than jumping multiple major versions ahead in one change.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Several upgraded actions (e.g., `actions/checkout@v6`, `actions/setup-python@v6`, `actions/cache@v5`, `github/codeql-action@v4`, `actions/github-script@v9`) don’t currently exist as published major versions; please confirm these tags are valid and adjust to the latest actually released major versions.
- For workflow stability, consider upgrading incrementally to the latest known major versions (e.g., from v3 to v4) and verifying each workflow run, rather than jumping multiple major versions ahead in one change.
## Individual Comments
### Comment 1
<location path=".github/workflows/docs.yml" line_range="15-22" />
<code_context>
steps:
- - uses: actions/checkout@v4
+ - uses: actions/checkout@v6
with:
fetch-depth: '2'
</code_context>
<issue_to_address>
**issue (bug_risk):** Referencing v6 of checkout/setup-python and v5 of cache may break if those major versions are not actually published yet.
These workflows now depend on checkout@v6, setup-python@v6, and cache@v5, which may not yet exist; if they’re unpublished, the jobs won’t start. Please pin to the latest confirmed majors (e.g., checkout@v4, setup-python@v5, cache@v4) and only bump once newer majors are available.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| uses: actions/checkout@v6 | ||
| - name: Set up Python | ||
| uses: actions/setup-python@v5 | ||
| uses: actions/setup-python@v6 | ||
| with: | ||
| python-version: '3.9' | ||
| cache: 'pip' | ||
| - name: Cache dependencies | ||
| uses: actions/cache@v4 | ||
| uses: actions/cache@v5 |
There was a problem hiding this comment.
issue (bug_risk): Referencing v6 of checkout/setup-python and v5 of cache may break if those major versions are not actually published yet.
These workflows now depend on checkout@v6, setup-python@v6, and cache@v5, which may not yet exist; if they’re unpublished, the jobs won’t start. Please pin to the latest confirmed majors (e.g., checkout@v4, setup-python@v5, cache@v4) and only bump once newer majors are available.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #45 +/- ##
=======================================
Coverage 82.43% 82.43%
=======================================
Files 21 21
Lines 1503 1503
Branches 190 190
=======================================
Hits 1239 1239
Misses 208 208
Partials 56 56 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Summary
Testing
Summary by Sourcery
Update GitHub workflow configurations to use newer, supported versions of core actions.
CI: