Overview
This guide walks you through the process of submitting a pull request (PR) to Minecraft Community Edition.Before You Start
Before creating a pull request:- Read the Contributing Guidelines
- Ensure your development environment is set up correctly
- Follow the Code Style Guide
- Test your changes thoroughly
Pull Request Template
When you create a PR, you’ll be prompted to fill out a template with the following sections:What does this PR do?
Provide a clear, concise description of what your PR accomplishes:Why?
Explain the motivation behind the changes:How to test
Provide clear steps for reviewers to test your changes:Screenshots / Videos
If your changes affect visuals or gameplay, include screenshots or videos:Testing Checklist
Before submitting your PR, verify the following:Builds without errors
Your code must compile successfully:
- Debug build succeeds
- Release build succeeds
- All platforms build (if applicable)
Tested in-game
Test your changes in the actual game:
- Feature works as expected
- No crashes or exceptions
- Performance is acceptable
- No visual glitches
- Multiplayer compatible (if applicable)
Submission Guidelines
Code Requirements
Original Work
Your PR must be your own work. Be prepared to provide proof of ownership if requested.
AI Usage
AI assistance for bug fixes is fine, but fully AI-generated implementations are not acceptable.
Code Quality
Follow the project’s code style and maintain high code quality standards.
Testing
Thoroughly test your changes before submitting.
Creating the Pull Request
Commit your changes
Make clear, atomic commits:Use descriptive commit messages that explain what and why.
Review Process
Once you submit your PR:What Reviewers Look For
- Functionality: Does it work as described?
- Code Quality: Is the code well-written and maintainable?
- Style Compliance: Does it follow the code style guide?
- Testing: Has it been adequately tested?
- Documentation: Are comments clear and helpful?
- Impact: Does it affect other parts of the codebase?
Addressing Feedback
When reviewers provide feedback:- Respond promptly: Address comments in a timely manner
- Ask questions: If feedback is unclear, ask for clarification
- Make changes: Update your code based on feedback
- Push updates: Push new commits to the same branch
- Mark resolved: Mark conversation threads as resolved when addressed
Automated Checks
Your PR may be subject to automated checks:- Build verification: Ensures code compiles
- Warning detection: Checks for new compiler warnings
- Code analysis: Static analysis for common issues
Merging Criteria
Your PR will be merged when:- ✅ All checklist items are completed
- ✅ Code builds without errors
- ✅ No new warnings introduced
- ✅ Tested in-game successfully
- ✅ Code review approved by maintainer(s)
- ✅ All automated checks pass
- ✅ Conflicts resolved (if any)
After Merging
Once your PR is merged:-
Delete your branch (optional but recommended):
-
Update your local repository:
- Celebrate: Your contribution is now part of the project!
Common Issues
Merge Conflicts
If your PR has conflicts:Failed Builds
If the build fails:- Check the build logs for errors
- Reproduce the build locally
- Fix the issues
- Push the fixes
Stale PRs
If your PR becomes stale:- Rebase on the latest main branch
- Verify everything still works
- Ping reviewers if needed
Best Practices
Keep PRs focused
Keep PRs focused
Each PR should address a single feature or bug. Avoid mixing unrelated changes.
Write good descriptions
Write good descriptions
Clear PR descriptions help reviewers understand your changes quickly.
Test thoroughly
Test thoroughly
The more testing you do, the faster your PR will be reviewed and merged.
Respond to feedback
Respond to feedback
Engage constructively with reviewers. Their goal is to help improve the code.
Be patient
Be patient
Reviewers are volunteers. It may take time to review your PR.
Questions?
If you have questions about the PR process:- Check existing PRs for examples
- Ask in the repository discussions
- Reach out to maintainers