Update Contributing.md for 1.9 name/repo changes

This commit is contained in:
Zach Brown 2016-03-15 12:51:03 -05:00
parent ad72109859
commit c8d7ee5b8a

View file

@ -1,27 +1,27 @@
Contributing to PaperSpigot Contributing to Paper
========================== ==========================
PaperSpigot has a very lenient policy towards PRs, but would prefer that you try and adhere to the following guidelines. PaperMC has a very lenient policy towards PRs, but would prefer that you try and adhere to the following guidelines.
## Understanding Patches ## Understanding Patches
Patches to PaperSpigot are very simple, but center around the directories 'PaperSpigot-API' and 'PaperSpigot-Server' Patches to Paper are very simple, but center around the directories 'Paper-API' and 'Paper-Server'
Assuming you already have forked the repository: Assuming you already have forked the repository:
1. Pull the latest changes from the main repository 1. Pull the latest changes from the main repository
2. Type `./applyPatches.sh` in git bash to apply the changes from upstream 2. Type `./applyPatches.sh` in git bash to apply the changes from upstream
3. cd into `PaperSpigot-Server` for server changes, and `PaperSpigot-API` for api changes 3. cd into `Paper-Server` for server changes, and `Paper-API` for api changes
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 PaperSpigot-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 PaperSpigot-Server/API but for PaperSpigot
- 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 PaperSpigot and Spigot don't
- If it says something like `212 commits ahead, 207 commits behind`, then type `git fetch` to update spigot/paperspigot - 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 PaperSpigot is very simple: Adding patches to Paper is very simple:
1) Modify `PaperSpigot-Server` and/or `PaperSpigot-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
@ -56,22 +56,22 @@ While we will fix minor formatting issues, you should stick to the guide below w
## Formatting ## Formatting
All modifications to non-PaperSpigot files should be marked All modifications to non-PaperSpigot files should be marked
- Multi line changes start with `// PaperSpigot start` and end with `// PaperSpigot 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: `// PaperSpigot 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
- Multi-line messages should start with `// PaperSpigot start` and use `/* Multi line message here */` for the message itself - Multi-line messages should start with `// Paper start` and use `/* Multi line message here */` for the message itself
- Single line changes should have `// PaperSpigot` or `// PaperSpigot - reason` - Single line changes should have `// Paper` or `// Paper - reason`
- For example: - For example:
````java ````java
entity.getWorld().dontbeStupid(); // PaperSpigot - was beStupid() which is bad entity.getWorld().dontbeStupid(); // Paper - was beStupid() which is bad
entity.getFriends().forEach(Entity::explode()); entity.getFriends().forEach(Entity::explode());
entity.a(); entity.a();
entity.b(); entity.b();
// PaperSpigot start - use plugin-set spawn // Paper start - use plugin-set spawn
// entity.getWorld().explode(entity.getWorld().getSpawn()); // entity.getWorld().explode(entity.getWorld().getSpawn());
Location spawnLocation = ((CraftWorld)entity.getWorld()).getSpawnLocation(); Location spawnLocation = ((CraftWorld)entity.getWorld()).getSpawnLocation();
entity.getWorld().explode(new BlockPosition(spawnLocation.getX(), spawnLocation.getY(), spawnLocation.getZ())); entity.getWorld().explode(new BlockPosition(spawnLocation.getX(), spawnLocation.getY(), spawnLocation.getZ()));
// PaperSpigot end // Paper end
```` ````
- We generally follow usual java style, or what is programmed into most IDEs and formatters by default - We generally follow usual java style, or what is programmed into most IDEs and formatters by default
- This is also known as oracle style - This is also known as oracle style