Review a pull request
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Dokumentation
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- github, javascript
- Bereich
- developer-experience
Rechercherichtung
Beginne mit der verlinkten GitHub-Anleitung zum Überprüfen von Änderungen an pull request, verwende anschließend die PR-Oberfläche von GitHub oder VSCode und checke den zugewiesenen PR mit gh pr checkout 123 aus. Führe den Code aus, vergleiche ihn mit den Akzeptanzkriterien des Tickets, stelle klärende Fragen und reiche das Review direkt im PR deines Teammitglieds ein.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Link to the coursework
Why are we doing this?
You must set aside a part of your development time to review the work of your team. In your technical interview, you may to asked to discuss a feature written by someone else in your team, so you will need to read and understand it.
How to review
- Use the GitHub PR interface, either on GitHub or via VSCode
- Checkout the PR locally
gh pr checkout 123and run the code to make sure it works - Test the code using the acceptance criteria on the ticket
- Ask clarifying questions to make sure you understand the code
- Offer any fixes or improvements you can see
Maximum time in hours
2
How to get help
Volunteers and other team members can help you with code review. You may choose to spend some time in class doing this together.
How to submit
Submit your review directly on your teammates PR.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 2
- Forks
- 62
- Ø Merge
- 2 T. 1 Std.
- Gemergte PRs (30 T.)
- 1
Beitragsleitfaden
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 CodeYourFuture/Checkpoint
-
📅 Sprint 1 Setup
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 15/100
CodeYourFuture/Checkpoint#77 ·
-
02 Prepare a demo Offen🏕 Priority Mandatory 📅 Sprint 2 Core
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 25/100
CodeYourFuture/Checkpoint#66 ·
-
🏕 Priority Mandatory 📅 Sprint 2 Core
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100
CodeYourFuture/Checkpoint#65 ·
-
🏕 Priority Mandatory 📅 Sprint 3 Core
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100
CodeYourFuture/Checkpoint#64 ·
-
🏕 Priority Mandatory 📅 Sprint 1 Core
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100
CodeYourFuture/Checkpoint#61 ·
Alle Issues in CodeYourFuture/Checkpoint
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
-
area-deployment area-integrations triage:bot-seen
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 86/100
-
Issue-Bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
sugarlabs/musicblocks#8924 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
ArduPilot/ardupilot_wiki#8088 ·
-
[BUG] createTool tools cannot be registered with Mastra when exactOptionalPropertyTypes is enabled Offencustomer-eng status: needs triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100