Skip to content

Instantly share code, notes, and snippets.

@EvilSeph
Last active December 15, 2015 03:19
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save EvilSeph/35bb477eaa1dffc5f1d7 to your computer and use it in GitHub Desktop.
Save EvilSeph/35bb477eaa1dffc5f1d7 to your computer and use it in GitHub Desktop.
Here's our Pull Request Format Template in a handy gist for easy copying.

Title

[PR Type] Brief summary. Fixes BUKKIT-####
PR Type can be B for Bukkit, C for CraftBukkit, B+C for a PR in both sides

Title Example:
[B+C] Provide an example commit for CONTRIBUTING.md. Fixes BUKKIT-1

Description:

The Issue:

Paragraphs explaining what the issue the PR is meant to be addressing.

Justification for this PR:

Paragraphs providing justification for the PR

PR Breakdown:

Paragraphs breaking down what the PR is doing, in detail.

Testing Results and Materials:

Paragraphs describing what you did to test the code in this PR and links to pre-compiled test binaries and source.

Relevant PR(s):

This should be links to accompanying PRs, or alternate PRs that attempted to perform the task. Each reference should have a reason attached as to why it is being referenced (for example: "Similar to PR ### but won't empty your Bukkits"). Accompanying PRs need no explanation, but still need to be linked.

B-#### - https://github.com/Bukkit/Bukkit/pull/#### - Reason
CB-#### - https://github.com/Bukkit/CraftBukkit/pull/#### - Reason

JIRA Ticket:

BUKKIT-#### - https://bukkit.atlassian.net/browse/BUKKIT-####

Pull Request Check List (For Your Use):

General:

  • Fits Bukkit's Goals
  • Leaky Ticket Ready
  • Code Meets Requirements
  • Code is Documented
  • Code Addresses Leaky Ticket
  • Followed Pull Request Format
  • Tested Code
  • Included Test Material and Source

If Applicable:

  • Importing New Minecraft Classes In Special Commit
  • Follows Minimal Diff Policy
  • Uses Proper CraftBukkit Comments
  • Imports Are Ordered, Separated and Organised Properly
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment