riseproject-dev / riseproject-dev/python-wheels
Narrow `pr-trigger.yml` to ignore publish-only changes in workflows
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 0
- Forks
- 0
- Merge moyen
- 16 h 14 min
- PR mergées (30 j)
- 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.
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le workflow pr-trigger.yml et comparez-le à build-numpy.yml, puis examinez la modification du workflow introduite par #169. Mettez à jour les conditions du déclencheur afin que les modifications concernant uniquement la publication ne lancent pas de builds, tandis que les autres modifications de scripts continuent de le faire, et vérifiez le comportement du workflow qui en résulte.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- github-actions
- Domaine
- build-system, ci-cd
- Type d'issue
- Fonctionnalité
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 68/100