PRs updates may cause unwanted PR creation in `docker-library/official-images`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Dockerfile
- Sterne
- 8.6k
- Forks
- 2k
- Ø Merge
- 10 Std. 19 Min.
- Gemergte PRs (30 T.)
- 16
Beschreibung
Situation
When Dockerfile-template.* changes are made and Dockerfile updates in the release line directories are also made, then the repo automatically triggers a PR to build a new release. The workflow official-pr is responsible for this.
Unless the update is part of a new release, then Dockerfile updates should generally not be done manually. These updates should be left to the automated processes that are triggered when the repo's automatic-updates.yml polling workflow detects a new Node.js release.
Suggestion
Research the background and add instructions to the CONTRIBUTING document about when Dockerfile updates should be made directly, and when it should be left to automation.
For those cases where no direct Dockerfile update should be included in a PR, describe also how to test such PRs.
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
Lies CONTRIBUTING.md zusammen mit .github/workflows/official-pr.yml und automatic-updates.yml, um zu verstehen, wann Änderungen an Dockerfile Pull Requests für offizielle Images auslösen. Aktualisiere CONTRIBUTING.md mit den Ausnahmen für direkte Aktualisierungen und Anweisungen zum Testen von Pull Requests, die keine Aktualisierung von Dockerfile enthalten sollen; als erledigt gilt die Aufgabe, wenn beide Fälle klar dokumentiert sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- docker, dockerfile, github-actions
- Bereich
- ci-cd, documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 72/100