Merge pull request #82 from willies952002/patch-1

Fix Errors in Contributing.md
This commit is contained in:
Zach 2016-03-15 15:42:44 -05:00
commit a4c39eb681

View file

@ -13,21 +13,21 @@ Assuming you already have forked the repository:
These directories aren't git repositories in the traditional sense: These directories aren't git repositories in the traditional sense:
- Every single commit in PaperSpigot-Server/API is a patch. - Every single commit in Paper-Server/API is a patch.
- 'origin/master' points to a directory similar to PaperSpigot-Server/API but for PaperSpigot - 'origin/master' points to a directory similar to Paper-Server/API but for Paper
- Typing `git status` should show that we are 10 or 11 commits ahead of master, meaning we have 10 or 11 patches that PaperSpigot and Spigot don't - Typing `git status` should show that we are 10 or 11 commits ahead of master, meaning we have 10 or 11 patches that Paper and Spigot don't
- If it says something like `212 commits ahead, 207 commits behind`, then type `git fetch` to update spigot/paper - If it says something like `212 commits ahead, 207 commits behind`, then type `git fetch` to update spigot/paper
## Adding Patches ## Adding Patches
Adding patches to Paper is very simple: Adding patches to Paper is very simple:
1) Modify `Paper-Server` and/or `Paper-API` with the appropriate changes 1. Modify `Paper-Server` and/or `Paper-API` with the appropriate changes
2) Type `git add .` to add your changes 2. Type `git add .` to add your changes
3) Run `git commit` with the desired patch message 3. Run `git commit` with the desired patch message
4) Run `./rebuildPatches.sh` in the main directory to convert your commit into a new patch 4. Run `./rebuildPatches.sh` in the main directory to convert your commit into a new patch
5) PR your patches back to this repository 5. PR your patches back to this repository
Your commit will be converted into a patch that you can then PR into PaperSpigot Your commit will be converted into a patch that you can then PR into Paper
## Modifying Patches ## Modifying Patches
Modifying previous patches is a bit more complex: Modifying previous patches is a bit more complex:
@ -55,7 +55,7 @@ We'll accept changes that make sense. You should be able to justify their existe
While we will fix minor formatting issues, you should stick to the guide below when making and submitting changes. While we will fix minor formatting issues, you should stick to the guide below when making and submitting changes.
## Formatting ## Formatting
All modifications to non-PaperSpigot files should be marked All modifications to non-Paper files should be marked
- Multi line changes start with `// Paper start` and end with `// Paper end` - Multi line changes start with `// Paper start` and end with `// Paper end`
- You can put a messages with a change if it isn't obvious, like this: `// Paper start - reason - You can put a messages with a change if it isn't obvious, like this: `// Paper start - reason
- Should generally be about the reason the change was made, what it was before, or what the change is - Should generally be about the reason the change was made, what it was before, or what the change is