ci: skip lint-and-test on non-code changes#53
Conversation
Use dorny/paths-filter to detect code-related file changes and skip the lint-and-test job for non-code PRs (docs, config, etc.). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 61f9d270f0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Coverage report
This PR does not seem to contain any modification to coverable code. |
examples/ contains executable Python covered by ruff check, so changes there should trigger lint-and-test. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e86a356f0f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
If the changes job fails (e.g. checkout or paths-filter error), lint-and-test is skipped and the label job could still run due to always(). Add explicit needs.changes.result == 'success' check. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 participant
Summary
changesjob usingdorny/paths-filter@v3to detect code-related file changeslint-and-testjob when only non-code files are modified (docs,.claude/, etc.)claude-review-and-labelto run even whenlint-and-testis skippedTest plan
lint-and-testis skippedlint-and-testruns normallyclaude-review-and-labelruns in both cases🤖 Generated with Claude Code