We welcome all contributions!
To get started, please feel free to start working on any issues we have created and that are not assigned to anyone yet. If you would like to pick up an issue, please leave a comment for the other contributors to see that this issue is already be worked on.
For starters, we recommend to pick up issues labelled good first issue.
In addition to everything written in here, there is a Collaborator Guide you can read into for additional information about collaborations.
- Fork the repository
- Clone your forked repository onto your local machine
- Inside your CLI (Command Line Interface), move into your working directory
- RUN
npm run docker-dev
if using docker ornpm run dev
otherwise.
Note: Initial setup will not require npm i or npm ci, as it is done in the npm run dev-setup
script.
Note: PR's containing commits that only updates package-lock.json files will not be accepted.
- For installing or updating packages, use "npm i" or "npm install".
- For reinstalling node_modules, run "npm ci". This ensures that the package-lock.json remains unchanged.
- Create a branch for your feature (see below for an example)
- Add your code
- Create a Pull Request
Below you can find an overview of the branches we are using.
- master (protected)
- dev (protected)
- documentation
- feature
-- feature/FEATURE-NAME
--- feature/#<Issue Number>_<little feature title>
--- feature/#<Issue Number>_<little feature title>
--- feature/#<Issue Number>_<little feature title>
- bugfix (fixing issues that are not urgent)
- hotfix (fixing issues that need to be merged ASAP)
An example for a ToDo list feature could look as following:
-- feature/#1_todo-list
For consistency and easier readability, we would like to ask everyone to use the templates for Pull Requests and Issues that we provide. They follow this pattern:
<subject> (A title to summarise what this is about)
<body> (A detailed description of your PR, feature idea or issue)
Resolves #ISSUE_NUMBER
Note: It is important to add the #number for the issue the PR is resolving in order to close the issue accordingly once the PR gets merged. You can read more about this here