Een pull-request indienen

De basis

Eerst moet u uw machine gereed maken om te kunnen ontwikkelen voor en met Sponge:

When you’re done and feel you’re ready for developing Sponge, decide which parts you want to work on.

Het schrijven van een PR

Het oplossen van bugs

Verduidelijk in een paar zinnen:

  • welke bug u heeft gevonden, en vooral

    • hoe het zich heeft gedragen

    • wat denk je wat er had moeten gebeuren

  • wat heb jij gefixed

  • hoe heb je het gefixed

Grote API toevoeging

So, you’ve developed a pretty large API addition that you want to now submit as a PR. Good! Constructive PR’s are what make this project keep getting better. Which brings us to writing the glorious PR description.

There have been a few in the past that go above and beyond the standards, examples include:

Of course, those examples are the extreme, but PR’s that are accepted and provide as a good standard of what should be included in a PR description are:

A few things that can be taken from this:

  • Links to any implementation PRs in clear view at the top of the PR, this can be achieved with GitHub Markdown

*SpongeAPI* | [SpongeCommon](html link) | [SpongeDocs](html link)
  • Clear description of what the API PR is aiming to do, this can be a brief summary as if writing an essay, at most 4 sentences long, depending on what the functionality is.

  • Clear code examples of how plugins can use the new feature (and if there are old features existing, why they needed to be changed).