-
Notifications
You must be signed in to change notification settings - Fork 183
Contributing
Daniel Evans edited this page Jun 5, 2016
·
24 revisions
If you are submitting a pull request for OpenRW:
- Include a reference to any issue the pull request is intended to address.
- Submit unrelated changes in different pull requests.
- If you aren't ready to merge your changes but are looking for feedback, include "[not ready]" in the title.
- Changes in behaviour should be shown to match the behaviour in the original game, if applicable.
- Ensure that the test suite still passes after your change, updating tests if required.
- Changes that increase compatibility with the game data.
- Fixes for technical issues, such as wide-screen or UHD resolution support.
- Feature enhancements, such as improved support for different input schemes.
- Changes where the ownership of the code is in question.
- Changes to support non-GTA titles.
When a pull request is submitted, at least two members of the organisation should have reviewed the changes before merging.