-
Notifications
You must be signed in to change notification settings - Fork 4
FAQ
This page answers common questions that are not covered in detail on other pages.
Ask for help. When asking, see the Troubleshooting page for what to include in your question. Being stuck is normal. Clear questions help others help you faster.
Another contributor should review your pull request before it is merged. Reviews help catch bugs, improve readability, and share knowledge across the team. See the Code review page for details.
Merge after:
- the pull request has been reviewed
- requested changes have been handled
- the linked issue is included in the PR description
- the change has been tested
- the team has approved merging
Do not merge directly to master without review.
- Switch back to
master. - Pull the latest changes.
- Delete your old branch.
- Pick a new issue if you want to continue.
git checkout master
git pull origin masterUsually, yes. Once the pull request is merged and the branch is no longer needed, it can be deleted. This keeps the repository clean. GitHub may offer a button to do this automatically.
Avoid unrelated changes. If you notice something else that should be fixed, open a separate issue or pull request. This makes review easier and keeps the project history clear.