[Actions] Only the relevant workflows should run after commits on a PR branch#257
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
This PR updates several GitHub Actions workflows to avoid running expensive generator/build jobs on every new commit to an existing PR when the latest push doesn’t touch the workflow’s relevant directories.
Changes:
- Added a
changesjob usingdorny/paths-filter@v3to detect whether the latest PR push (synchronize) touched relevant paths. - Gated key downstream jobs (e.g.,
Thunder,generate_lua,prepare) usingneeds: [changes]and conditionalif:logic to skip when irrelevant.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| .github/workflows/ProxyStubGenerator.yml | Adds a changes job and gates the Thunder reusable workflow based on incremental PR changes. |
| .github/workflows/PluginSkeletonGenerator.yml | Adds a changes job and gates Thunder and prepare so generator work only runs when latest PR push is relevant. |
| .github/workflows/LuaGenerator.yml | Adds a changes job and gates generate_lua to run only when the latest PR push affects Lua/ProxyStub-related paths. |
| .github/workflows/JsonGenerator.yml | Adds a changes job and gates Thunder based on incremental PR changes. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
LuaGenerator ResultsNo changes detected. |
|
Plugin skeleton preview (latest run): No changes detected in generated plugin code. The PR changes are internal to the generator or do not affect output. |
LuaGenerator ResultsNo changes detected. |
ProxyStubGenerator ResultsNo changes detected. |
JsonGenerator ResultsNo changes detected. |
|
Plugin skeleton preview (latest run): No changes detected in generated plugin code. The PR changes are internal to the generator or do not affect output. |
ProxyStubGenerator ResultsNo changes detected. |
JsonGenerator ResultsNo changes detected. |
No description provided.