Add a note for copilot-setup-steps.yml and using with a merge queue
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 84/100
- Issue-Typ
- Dokumentation
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- github-actions
- Bereich
- ci-cd, documentation
Rechercherichtung
Beginne mit dem verlinkten Artikel zu den Copilot-Setup-Schritten und finde das vorhandene copilot-setup-steps.yml-Beispiel. Überprüfe, wie sich dessen push- und pull_request-Trigger bei Merge-Queue-Branches verhalten, und aktualisiere das Beispiel oder füge einen Inline-Hinweis mit der vorgeschlagenen Konfiguration hinzu. Als erledigt gilt die Aufgabe, wenn die Dokumentation vor unnötigen Merge-Queue-Läufen warnt und gleichzeitig manuelle Tests und die Validierung von Änderungen ermöglicht.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Code of Conduct
- I have read and agree to the GitHub Docs project's Code of Conduct
What article on docs.github.com is affected?
What part(s) of the article would you like to see updated?
Somewhat related to #42379
In the github docs, it says that you can add the following to you copilot-setup-steps.yml file:
name: "Copilot Setup Steps"
## Background
# Automatically run the setup steps when they are changed to allow for easy validation, and
# allow manual testing through the repository's "Actions" tab
on:
workflow_dispatch:
push:
paths:
- .github/workflows/copilot-setup-steps.yml
pull_request:
paths:
- .github/workflows/copilot-setup-steps.yml
While this works, I've found that if you use a merge queue then using the above configuration results in your workflow running every time a PR is added to the merge queue. We are use the larger, custom image GitHub hosted runners and this results in unnecessary runs & costs.
github/copilot-sdk was the first public GitHub repo I found that had the same issue(although this one is just using default ubuntu-latest runners). It helped me figure out that behavior wasn't unique to our setup
Proposed Content Update
I'd like to update the docs to either add an inline comment with commented out code or add a little inline note about if using a merge queue. This is what ended up working for us:
name: "Copilot Setup Steps"
# Automatically run when changed so setup issues surface immediately.
# Also allows manual testing via the "Actions" tab.
on:
workflow_dispatch:
push:
branches-ignore:
- 'gh-readonly-queue/**'
paths:
- .github/workflows/copilot-setup-steps.yml
pull_request:
branches:
- main
paths:
- .github/workflows/copilot-setup-steps.yml
Additional information
No response
- Vorherrschende Sprache
- TypeScript
- Sterne
- 20.9k
- Forks
- 68.9k
- Ø Merge
- 8 Std. 3 Min.
- Gemergte PRs (30 T.)
- 118
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus github/docs
-
content driver persona never-stale
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
github/docs#46116 · 4 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
localization never-stale
Schwierigkeit 2/5 1-2 Tage Anfängerfreundlichkeit 72/100
github/docs#45949 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
content localization never-stale
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
github/docs#45858 · 2 Kommentare · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
content localization never-stale
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
github/docs#45773 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
content dependabot needs SME
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
github/docs#45681 · 6 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
aiko-chan-ai/DiscordBotClient#380 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
vercel/ai-elements#507 ·
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 84/100
anaclumos/qa-interns#148 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag