7.6 KiB
Wie man einen Pull-Request (PR) öffnet
Ein Pull-Request (PR) ermöglicht es dir, Änderungen von deinem Fork auf GitHub an FreeCodeCamp.orgs Hauptrepository zu senden. Wenn du mit den Änderungen am Code fertig bist, kannst du diese Richtlinien befolgen, um einen PR zu eröffnen.
[!NOTE] Dein PR sollte in Englisch verfasst sein. Schaue hier nach, wie du Übersetzungen beisteuern kannst.
Bereite einen guten PR-Titel vor
Wir empfehlen, konventionelle Titel und Nachrichten für Commits und Pull-Requests zu verwenden. Die Konvention hat das folgende Format:
<type>([optional scope(s)]): <description>
Zum Beispiel:
fix(learn): tests for the do...while loop challenge
Wenn du einen Pull-Request (PR) öffnest, kannst du den Typ, den Geltungsbereich [scope] (optional) und die Beschreibung wie folgt festlegen.
Typ:
Typ | Wann wählen |
---|---|
fix | Geänderte oder aktualisierte/verbesserte Funktionalität, Tests, der Wortlaut einer Lektion, usw. |
feat | Nur wenn du neue Funktionen, Tests usw. hinzufügst. |
chore | Änderungen, die sich nicht auf den Code, die Tests oder den Wortlaut einer Lektion beziehen. |
docs | Änderungen im Verzeichnis /docs oder in den Mitwirkungsrichtlinien, etc. |
Geltungsbereich (Scope):
Du kannst einen Geltungsbereich aus dieser Liste von Labels auswählen.
Beschreibung:
Halte sie kurz (weniger als 30 Zeichen) und einfach. Du kannst weitere Informationen im PR-Beschreibungsfeld und in den Kommentaren hinzufügen.
Einige Beispiele für gute PR-Titel wären:
fix(a11y): improved search bar contrast
feat: add more tests to HTML and CSS challenges
fix(api,client): prevent CORS errors on form submission
docs(i18n): Chinese translation of local setup
Einen Pull-Request vorschlagen
-
Sobald die Änderungen übertragen wurden, wirst du aufgefordert, einen Pull-Request auf der GitHub-Seite deines Forks zu erstellen.
-
Grundsätzlich sollten alle Pull-Requests gegen das Haupt-Repository von freeCodeCamp, den
main
-Branch, gerichtet sein.Stelle sicher, dass dein Base Fork auf freeCodeCamp/freeCodeCamp eingestellt ist, wenn du einen Pull-Request einreichst.
-
Übermittle den Pull-Request von deinem Branch an den
main
-Branch von freeCodeCamp. -
Im Hauptteil deines PR fügst du eine ausführlichere Zusammenfassung der Änderungen ein, die du vorgenommen hast und warum.
-
Du erhältst eine Vorlage für einen Pull-Request. Dies ist eine Checkliste, die du befolgen solltest, bevor du den Pull-Request öffnest.
-
Fülle die Details so aus, wie du es für richtig hältst. Diese Informationen werden geprüft und die Prüfer entscheiden, ob dein Pull-Request angenommen wird oder nicht.
-
Wenn der PR ein bestehendes GitHub Problem beheben soll, dann am Ende von der Beschreibungstext deines PR verwenden Sie das Schlüsselwort Schließt mit der Ticketnummer zu automatisch schließen, wenn der PR akzeptiert und zusammengeführt wird.
Beispiel:
Closes #123
wird Issue 123 schließen
-
-
Gib an, ob du auf einer lokalen Kopie der Website getestet hast oder nicht.
-
Das ist sehr wichtig, wenn du Änderungen vornimmst, die nicht nur Textinhalte wie die Dokumentation oder eine Aufgabenbeschreibung betreffen. Beispiele für Änderungen, die lokal getestet werden müssen, sind JavaScript, CSS oder HTML, die die Funktionalität oder das Layout einer Seite verändern könnten.
-
Wenn dein PR das Verhalten einer Seite beeinflusst, sollte er von entsprechenden Cypress Integrationstests begleitet werden.
-
Feedback zu Pull-Requests
🎉 Congratulations on making a PR and thanks a lot for taking the time to contribute.
Unsere Moderatoren werden jetzt einen Blick darauf werfen und dir ein Feedback hinterlassen. Bitte habe Geduld mit den anderen Moderatoren und respektiere ihre Zeit. Alle Pull-Requests werden zu gegebener Zeit überprüft.
Und wie immer kannst du Fragen in der Kategorie 'Contributors' in unserem Forum oder im Chatraum für Mitwirkende stellen.
[!TIP] Wenn du mehr Pull-Requests beisteuern willst, empfehlen wir dir, die Richtlinien für Änderungen und Synchronisierung zu lesen, damit du deinen Fork nicht löschen musst.
Konflikte bei Pull-Requests
Es kann zu Konflikten kommen, weil viele Mitwirkende an dem Repository arbeiten und Änderungen deinen PR zerstören können, der noch auf eine Überprüfung und Zusammenführung wartet.
In den meisten Fällen brauchst du keinen Rebase, da wir alle Commits vernichten. Wenn jedoch ein Rebase verlangt wird, solltest du wie folgt vorgehen.
Für die üblichen Fehlerbehebungen und Funktionen
Wenn du an regulären Bugs und Features auf unserem Entwicklungszweig main
arbeitest, kannst du einen einfachen Rebase durchführen:
-
Rebase deiner lokalen Kopie:
git checkout <pr-branch> git pull --rebase upstream main
-
Löse alle Konflikte und füge Commits hinzu / bzw. bearbeite sie
# Either git add . git commit -m "chore: resolve conflicts" # Or git add . git commit --amend --no-edit
-
Schiebe deine Änderungen in den PR zurück
git push --force origin <pr-branch>
Für anstehende Studienpläne und Features
Wenn du an Funktionen für unseren kommenden next-*
-Branch arbeitest, musst du Rosinenpickerei betreiben:
-
Achte darauf, dass dein Upstream mit deinem Local übereinstimmt:
git checkout main git fetch --all --prune git checkout next-python-projects git reset --hard upstream/next-python-projects
-
Backup erstellen
a. Entweder löschst du deinen lokalen Branch, nachdem du ein Backup gemacht hast (wenn du ihn noch lokal hast):
git checkout <pr-branch-name> # Beispiel: # git checkout feat/add-numpy-video-question git checkout -b <backup-branch-name> # Beispiel: # git checkout -b backup-feat/add-numpy-video-question git branch -D <pr-branch-name>
b. Oder einfach ein Backup deines PR-Branch (wenn du ihn nicht lokal hast):
git checkout -b <backup-branch-name> origin/<pr-branch-name> # Beispiel: # git checkout -b backup-feat/add-numpy-video-question origin/feat/add-numpy-video-question
-
Beginne mit einer weißen Weste:
git checkout -b <pr-branch-name> next-python-projects git cherry-pick <commit-hash>
-
Beseitige alle Konflikte und räume auf, installiere Tests und führe sie durch
npm run clean npm ci npm run test:curriculum --superblock=<superblock-name> # Beispiel: # npm run test:curriculum --superblock=python-for-everyone
-
Wenn alles gut aussieht, schickst du es zurück an den PR
git push --force origin <pr-branch-name>