docs: Details in CONTRIBUTING
Signed-off-by: Tibo De Peuter <tibo.depeuter@ugent.be>
This commit is contained in:
parent
2030b97c21
commit
6f4e547c9e
1 changed files with 42 additions and 25 deletions
|
@ -1,24 +1,33 @@
|
||||||
# Hoe bijdragen aan Dwengo-1?
|
# Hoe bijdragen aan Dwengo-1?
|
||||||
|
|
||||||
|
Bedankt dat je wil bijdragen aan Dwengo-1!
|
||||||
|
Hieronder vind je enkele richtlijnen om je op weg te helpen.
|
||||||
|
|
||||||
|
Over het algemeen bestaat de workflow uit de volgende stappen:
|
||||||
|
|
||||||
|
1. [Een issue aanmaken](#issues)
|
||||||
|
2. [Een branch maken](#workflow)
|
||||||
|
3. [Code schrijven](#coding-conventions)
|
||||||
|
4. [Werk committen](#commits)
|
||||||
|
5. [Een pull request maken](#pull-request)
|
||||||
|
|
||||||
## Issues
|
## Issues
|
||||||
|
|
||||||
Maak gebruik van de [label set](https://github.com/SELab-2/Dwengo-1/labels).
|
Als je een issue aanmaakt is het belangrijk om zo veel mogelijk (relevante) informatie te geven.
|
||||||
|
Om je op weg te helpen zijn er [templates](.github/ISSUE_TEMPLATE) voorzien.
|
||||||
|
Gebruik deze om alle nodige informatie te verzamelen.
|
||||||
|
|
||||||
Voor bug reports:
|
Gebruik de juiste [labels](https://github.com/SELab-2/Dwengo-1/labels) om te helpen een onderscheid te maken tussen verschillende categorieën issues.
|
||||||
|
|
||||||
Geef zo veel mogelijk informatie. Als er error berichten zijn, graag in tekst bijvoegen. Geen screenshots van error
|
Ken jezelf toe aan een issue als je eraan werkt, zodat er beter een overzicht bewaard kan worden.
|
||||||
messages, enkel van visuele bugs.
|
Op die manier vermijd je onnodig werk.
|
||||||
|
|
||||||
Ken jezelf toe aan een issue als je eraan werkt, zodat iedereen een overzicht heeft van waar aan gewerkt wordt en door
|
|
||||||
wie. Zo wordt onnodig werk vermeden.
|
|
||||||
|
|
||||||
## Workflow
|
## Workflow
|
||||||
|
|
||||||
We zullen [Gitflow](https://www.atlassian.com/git/tutorials/comparing-workflows/gitflow-workflow) gebruiken
|
We zullen (verzachte versie van) [Gitflow](https://www.atlassian.com/git/tutorials/comparing-workflows/gitflow-workflow) gebruiken.
|
||||||
|
Lees [hier](TODO-wiki) meer over deze beslissing.
|
||||||
|
|
||||||
Lees [hier](wiki) meer over deze beslissing
|
Concreet betekent dit dat het project uit de volgende branches bestaat:
|
||||||
|
|
||||||
Concreet:
|
|
||||||
|
|
||||||
- `main`
|
- `main`
|
||||||
- Incl. tags (`v1.2.3`)
|
- Incl. tags (`v1.2.3`)
|
||||||
|
@ -28,8 +37,23 @@ Concreet:
|
||||||
- `fix/something-#x`: Voor (minder dringende) bug fixes. Bug fixes worden aan een issue gelinkt.
|
- `fix/something-#x`: Voor (minder dringende) bug fixes. Bug fixes worden aan een issue gelinkt.
|
||||||
- `release/x.y.z`: Release prep branch
|
- `release/x.y.z`: Release prep branch
|
||||||
|
|
||||||
|
## Coding conventions
|
||||||
|
|
||||||
|
- Formatting: [Prettier](https://prettier.io/)
|
||||||
|
- Linting: Maak gebruik van [ESLint](https://typescript-eslint.io/) of aan de hand van de [
|
||||||
|
`npm` commando's](package.json).
|
||||||
|
|
||||||
|
Voel je vrij om zelf commit hooks te installeren, maar we dwingen dit niet af.
|
||||||
|
|
||||||
## Commits
|
## Commits
|
||||||
|
|
||||||
|
Om de geschiedenis van het project overzichtelijk te houden, maken we gebruik van [conventional commits](https://www.conventionalcommits.org/).
|
||||||
|
|
||||||
|
|
||||||
|
Concreet
|
||||||
|
|
||||||
|
Dit betekent dat elke commit een duidelijke boodschap moet hebben, die volgens een bepaald formaat is opgesteld.
|
||||||
|
|
||||||
Maken gebruik van [conventional commits](https://www.conventionalcommits.org/)
|
Maken gebruik van [conventional commits](https://www.conventionalcommits.org/)
|
||||||
|
|
||||||
Lees [hier](wiki) meer over deze beslissing
|
Lees [hier](wiki) meer over deze beslissing
|
||||||
|
@ -46,26 +70,19 @@ type options:
|
||||||
Als je een commit 'fixt', gebruik dan [
|
Als je een commit 'fixt', gebruik dan [
|
||||||
`git commit --fixup`](https://git-scm.com/docs/git-commit#Documentation/git-commit.txt---fixupamendrewordltcommitgt)
|
`git commit --fixup`](https://git-scm.com/docs/git-commit#Documentation/git-commit.txt---fixupamendrewordltcommitgt)
|
||||||
|
|
||||||
Als je een commit niet alleen hebt geschreven, maak dan
|
Als je een commit niet alleen hebt geschreven, maak dan een [commit met meerdere auteurs](https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors).
|
||||||
een [commit met meerdere auteurs](https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors).
|
|
||||||
|
|
||||||
## Pull request...
|
## Pull request
|
||||||
|
|
||||||
|
Eens je code hebt geschreven en gecommit, is het tijd om een pull request te maken.
|
||||||
|
Het is fijn als je meteen ([draft](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/about-pull-requests#draft-pull-requests)) pull requests maakt, zodat anderen kunnen meekijken en feedback kunnen geven.
|
||||||
|
|
||||||
Als je aan visuele features werkt, voeg dan een screenshot van de omgeving van de feature toe, voor en nadat de feature
|
Als je aan visuele features werkt, voeg dan een screenshot van de omgeving van de feature toe, voor en nadat de feature
|
||||||
geïmplementeerd werd.
|
geïmplementeerd werd.
|
||||||
|
|
||||||
Start een draft pull request vanaf je een nieuwe feature branch pusht naar de server.
|
Je zult merken dat sommige branches [beschermd](https://docs.github.com/en/github/administering-a-repository/about-protected-branches) zijn.
|
||||||
|
Dit betekent dat je niet zomaar kan mergen naar deze branches:
|
||||||
Policies
|
|
||||||
|
|
||||||
- naar `main`: kan enkel vanuit `release/x.y.z`
|
- naar `main`: kan enkel vanuit `release/x.y.z`
|
||||||
- naar `dev`: wordt nagekeken alvorens te mergen
|
- naar `dev`: wordt nagekeken alvorens te mergen
|
||||||
- elders: vrije keuze
|
- elders: vrije keuze
|
||||||
|
|
||||||
## Coding conventions
|
|
||||||
|
|
||||||
- Formatting: [Prettier](https://prettier.io/)
|
|
||||||
- Linting: Maak gebruik van [ESLint](https://typescript-eslint.io/) of aan de hand van de [
|
|
||||||
`npm` commando's](package.json).
|
|
||||||
|
|
||||||
Voel je vrij om zelf commit hooks te installeren, maar we dwingen dit niet af.
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue