A pull request is a chance to look at your work with fresh eyes before it becomes part of main. In this tutorial, you'll review your own changes, apply improvements, and merge the first feature for your website.
Prerequisites
- An open pull request for your
add-starred-listbranch. If you haven't opened one yet, see Writing and storing your code.
Why review code?
Reviewing changes before you merge helps you catch bugs, spot missing error handling, and improve readability while the work is still fresh. Even when you work alone, a deliberate review builds a habit that keeps your software projects healthy and makes collaboration easier later.
Reviewing your changes in the pull request
The Files changed tab in a pull request on GitHub shows a diff of everything your branch adds or modifies. Review it carefully.
- Click View your pull requests to go to your pull requests inbox on GitHub. Alternatively, navigate to your
stargazers-logrepository and click the Pull requests tab. - Click on the title to open your pull request for the
add-starred-listbranch. - Click the Files changed tab.
- Read the diff for each file, looking for:
- Bugs or logic errors, such as a mismatched file name in
script.js. - Missing error handling, such as what happens if
events.jsonfails to load. - Accessibility issues, such as missing text alternatives or unclear labels.
- Anything unclear that a future reader might struggle with.
- Bugs or logic errors, such as a mismatched file name in
- To note something you want to change, hover over a line, click , type a comment, then click Comment.
Optional: Getting an AI second opinion
You can also ask 副驾驶聊天 in your editor to review the same changes and offer a second perspective.
-
In your editor, open 副驾驶聊天.
-
Ask it to review your changes with a prompt like the following.
Text Review my changes for bugs, missing error handling, and accessibility issues.
Review my changes for bugs, missing error handling, and accessibility issues. -
Read the suggestions and decide which ones improve your feature. You're always in control of which changes to make.
提示
If you enjoyed using Copilot to review your own code, you can sign up for a paid plan to get additional AI credits and Copilot 代码评审, which can review your pull requests automatically when you add Copilot as a reviewer.
For more information, see 为自己设置GitHub Copilot and 关于GitHub Copilot 代码评审 in the GitHub Enterprise Cloud documentation.
Applying feedback
Turn your review notes and any 副驾驶聊天 suggestions you agree with into updates.
- In your editor, update the affected files to address what you found.
- In GitHub Desktop, enter a commit message such as
Address review feedback, then click Commit # files to add-starred-list. - Click Push origin to update your pull request with the new commit.
Merging the pull request
Once you're satisfied with your changes, merge them into main.
- On GitHub, return to your pull request and refresh the page.
- Click the Conversation tab.
- Click Merge pull request, then click Confirm merge.
- Click Delete branch to clean up your
add-starred-listbranch.
Checking your progress
Update your project board to reflect the completed work.
- On GitHub, navigate to your
Stargazers logproject board from the Projects tab in your repository. - Drag the
Display a list of starred repositoriesitem from In Progress to Done.
If you have more features to build, create new issues and move them to In Progress when you start the next cycle.
The complete workflow
You've now run through the full developer workflow:
- Planned the work with an issue and a project board.
- Created a branch and built your feature.
- Opened a pull request.
- Reviewed your own changes (optionally with help from Copilot).
- Merged your feature into
main. - Updated your project board to reflect your progress.
What you accomplished
| Task | Outcome |
|---|---|
| Reviewed changes in the Files changed tab | You opened the pull request's Files changed tab and read the diff. |
| Applied feedback and got an optional AI assist | You applied changes, and possibly asked 副驾驶聊天 for a review. |
| Merged your feature | You merged your changes into main. |
| Updated your progress | You moved the issue to Done on your project board. |
Next steps
- Publish your software project to a live URL that updates your website automatically. Continue to Deploying your website automatically.