[TECH] Lancer la CI des PRs au statut draft #10444
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
yannbertrand
added
👀 Tech Review Needed
cross-team
|
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
| @@ -12,8 +12,6 @@ jobs: | |||
|
|
|||
| steps: | |||
| - name: Trigger CircleCI | |||
| # ensure PR is not draft | |||
| if: '! github.event.pull_request.draft' | |||
There was a problem hiding this comment.
Je préférerai un filtre sur le label team-devcomp ou autres pour votre usage non ? 🤔
ça a plutôt était éprouvé que la consommation avait été réduite
There was a problem hiding this comment.
La conso a aussi réduite car chaque push lançait la CI même si il n'y avait pas de PR ! La proposition ici est un entre deux temporaire
HEYGUL
approved these changes
Oct 30, 2024
VincentHardouin
approved these changes
Oct 30, 2024
yannbertrand
added
Tech Review OK
🚀 Ready to Merge
and removed
👀 Tech Review Needed
labels
pix-service-auto-merge
force-pushed
the
tech-trigger-draft-prs
branch
from
a6dba30 to
e0aef9f
Compare
pix-service-auto-merge
force-pushed
the
tech-trigger-draft-prs
branch
from
e0aef9f to
abd3e5d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🍂 Problème
Depuis #10133, on limite les déclenchements de CI pour diminuer notre consommation. L'équipe DevComp métier a l'habitude d'utiliser le statut de PR "Draft" pour indiquer qu'une PR est toujours en cours de construction mais nécessite tout de même une boucle de feedback de tests principalement pour valider techniquement les modifications de contenu (exemple dans #10423).
🌰 Proposition
On propose pendant un mois de relancer la CI sur les PRs au statut "Draft" et de voir l'incidence sur la consommation.
🎃 Remarques
Il y a un sujet plus long terme suivi côté DevComp de ne pas lancer l'intégralité des tests du monorepo lors des modifications de contenu Modulix. Mais nous n'avons pas identifié d'alternative simple à mettre en place.
🪵 Pour tester
Cette PR a été ouverte en "Draft" pour vérifier que la CI est bien déclenchée. => Le vérifier via les logs GhA et CircleCI
Elle est ensuite passée en "Ready for review".