nodejs / nodejs/node-core-utils
Allow `request-ci` on the collaborator's PR without restriction of at least 1 approval
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 314
- Forks
- 137
- Merge moyen
- 2 j 8 h
- PR mergées (30 j)
- 24
Description
IIRC collaborators are able to manually trigger CI through Jenkins without needing 1 approval on the PR, maybe it makes sense for the collaborators to use request-ci label on their own or other collaborators's PR without at least 1 approval restriction.
https://github.com/nodejs/node-core-utils/blob/a68af4b7744e8416bfc0f062570688b67d5e10dc/lib/pr_checker.js#L560 just an idea - could we check either the PR author is a current collaborator?
Guide de contribution
Ouvrir le guide de contribution
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 dans lib/pr_checker.js vers la ligne 560 et suivez la restriction d’approbation utilisée par le label request-ci. Vérifiez comment sont représentés le statut de collaborator et l’auteur des pull requests, puis vérifiez les contrôles CI-label pertinents. C’est terminé lorsqu’un collaborator éligible peut utiliser request-ci sur sa propre pull request ou celle d’un autre collaborator sans nécessiter une approbation.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript
- Domaine
- authorization, ci-cd
- Type d'issue
- Fonctionnalité
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 56/100