PRs management
When handling pull requests, maintainers may need to synchronize contributor forks:
bash
git clone "[email protected]:USER/FORK" "dest_dir"
cd dest_dir
git remote add upstream "[email protected]:ThePorgs/REPO"
git fetch upstream
git checkout "TARGET_FORK_BRANCH"
git merge --no-edit upstream/"ORIGIN_BRANCH"
# Resolve conflicts if any
git pushExternal contributor PRs targeting dev are squash-merged. Release merges of dev → main (images) use a merge commit instead; see Images.
Review guidelines
Code quality
- Check for proper error handling
- Verify type hints and documentation
- Ensure backward compatibility
- Review test coverage
Documentation
- Verify documentation updates
- Check for proper formatting
- Ensure all new features are documented
- Validate code examples
Security
- Review for potential security issues
- Check for proper input validation
- Verify authentication mechanisms
- Ensure no sensitive data is exposed
Performance
- Check for resource usage
- Verify build times
- Review memory consumption
- Test on different architectures