nodejs / nodejs/node-core-utils
Branch protection improvements
Nessuno ha ancora preso questa issue.
- Lingua principale
- JavaScript
- Stelle
- 314
- Fork
- 137
- Merge medio
- 2g 8h
- PR unite (30g)
- 24
Descrizione
FYI I just created a ruleset for this repo: https://github.com/nodejs/node-core-utils/settings/rules/4654889
It targets the default branch and enables:
- Restrict deletions
- Require linear history
- Require a pull request with at least one approval
- Block force pushes
Only the pull request requirements are new compared to the old branch protection rule (https://github.com/nodejs/node-core-utils/settings/branch_protection_rules/17880641)
I think we can delete that rule. The only thing that remains in it is the restriction of who can push and that only restricts the nodejs-github-bot account. I don't think there's a convenient way to port that to rulesets. We could use the bypass list but then every pull request would always be red and require collaborators to click the bypass checkbox.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Esamina il ruleset nella pagina collegata delle impostazioni del repository GitHub e confrontalo con la regola di protezione del branch esistente. Conferma se la restrizione di push di nodejs-github-bot della vecchia regola può essere sostituita o rimossa senza compromettere le pull request. Il lavoro è completato quando il repository dispone dell’unica configurazione di protezione del branch prevista.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- github
- Ambito
- devops
- Tipo di issue
- Funzionalità
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 30/100