riseproject-dev / riseproject-dev/python-wheels
Narrow `pr-trigger.yml` to ignore publish-only changes in workflows
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 0
- Forks
- 0
- Avg merge
- 16h 14m
- Merged PRs (30d)
- 952
Description
After creating #169 , the build-numpy.yml workflow kicked off again despite no new version to build. This is expected due to how the existing checker workflows were designed, and the publish step wasn't going to run anyway since it wasn't on main, but it's wasteful to be running a build in this case. If it's simple enough, we should change the pr-trigger.yml workflow so that it only triggers builds when parts of the script other than the publish job are modified. This may be too complex to justify.
Alternatively, we could do some further rework (once the composite action in #169 is merged) to have it perform the publish steps in a "mock" mode, so that we're testing the publishing steps as much as possible too. It would mean letting the superfluous wheel builds to run though, without further tweaks.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the pr-trigger.yml workflow and compare it with build-numpy.yml, then review the workflow change introduced by #169. Update the trigger conditions so publish-only changes do not start builds while other script changes still do, and verify the resulting workflow behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github-actions
- Domain
- build-system, ci-cd
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100