Richtlijnen voor het bijdragen
Er zal altijd vraag zijn naar ontwikkelaars om ons te helpen met het verbeteren van SpongeAPI. Er is niet zoiets als een perfect project en er kan altijd wel wat verbeterd worden. Als je een geïnteresseerde ontwikkelaar bent, twijfel dan geen moment en help ons. Zorg er wel voor dat je onze richtlijnen volgt.
Notitie
Ontwikkelaars die vastberadenheid en consistentie in hun bijdrages aan het project tonen kunnen door de team leiders uitgenodigd worden om toe te treden tot het Sponge team. Er is geen formele aanvraagprocedure. Wij vragen u en niet andersom.
Algemene stappen
Maak je werkomgeving klaar zoals beschreven in Voorberedingen voor Ontwikkeling.
Zorg ervoor dat u bekend bent met Git en GitHub. Mocht u uw kennis willen opfrissen dan kan dat hier: Hoe u Git(Hub) moet gebruiken
Check for existing issues in the SpongeAPI, SpongeCommon and SpongeDocs repositories. There is possibly someone else already working on the same thing. You can also check issues marked with “help wanted” for existing issues we need your help with.
Notitie
Please don’t submit pull requests for small, non-functional changes such as fixing typos or renaming variable names. Instead, you can:
join #spongedev on IRC (irc.esper.net) and we’ll change it together with the other smaller changes.
Join our Discord Server and post on the
#dev
channel. Discord and IRC are linked, so don’t worry. We won’t miss your message!You can also add it to our list of Minor Issues on GitHub.
If the issue requires a bigger change you may want to submit the issues without the necessary changes first, so we can confirm the issue and know that you’re working on fixing it. You should also create a draft pull request or comment with
~wip
so we can already start reviewing them.‘Fork’ het project, kopieer het en maak je veranderingen in een extra branch/zijtak.
Test jouw aanpassingen (let er op dat het correct compileert) en verstuur de wijzigingen naar jouw fork.
Bevestig de pull request met een klein beschrijving met daarin wat je veranderd hebt, en waarom het op die manier zou moeten worden veranderd
Als je extra veranderingen maakt, push de nieuwe commits dan naar jou branch. squash je verandering niet!, dat maakt het heel erg moeilijk voor ons om te zien wat je hebt veranderd in vergelijking met een eerdere versie van je pull request.
Zorg ervoor dat uw PR gerebased is aan de laatste veranderingen van de branch waar u van plan bent mee te mergen. Als u hulp nodig heeft met het rebasen, vraag het ons!
Tip
Als u twijfelt op welke branch u uw werk moet baseren, lees onze Repository Branch Layout alvorens uw PR in te dienen.