nodejs / nodejs/node-core-utils
Allow `request-ci` on the collaborator's PR without restriction of at least 1 approval
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 314
- Forks
- 137
- Ø Merge
- 2 T. 8 Std.
- Gemergte PRs (30 T.)
- 24
Beschreibung
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?
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in lib/pr_checker.js ungefähr bei Zeile 560 und verfolge die für das Label request-ci verwendete Genehmigungsbeschränkung. Prüfe, wie der Status von Collaborators und die Urheberschaft von Pull Requests dargestellt werden, und überprüfe anschließend die relevanten Prüfungen für das CI-Label. Erledigt ist die Aufgabe, wenn ein berechtigter Collaborator request-ci für den eigenen oder den Pull Request eines anderen Collaborators verwenden kann, ohne eine Genehmigung zu benötigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- authorization, ci-cd
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 56/100