The branch main
is a central development branch, which should be clean and ready for production. There should be no unnecessary comments and debugging code.
-
Please follow the guide on how to work on new feature
-
Make a commit to your feature branch. Commit messages should be descriptive and help others to understand the change that was made.
- Make sure the code is following the Coding standards
-
When your feature is ready to merge, create a pull request.
- Before merging, please make sure that the code is following the Coding standards by running
npm run lint
. Make sure no errors are reported. - This will automatically assign a reviewer to your pull request.
- There are several test and review that needs to be passed before merging.
- Before merging, please make sure that the code is following the Coding standards by running
-
Tidy up! After the pull request is merged delete the branch from your local and GitHub repository.
git branch -D <your-branch-name>
git push origin --delete your-branch-name
Before working on a feature, please make sure that you're on the latest version of the main branch of the repository.
git fetch origin main:main
git checkout -b <newFeature> main
To ensure update with the development dependencies of the project run
npm install
Then, you can start working on the feature.
code formatting rules are defined in .eslintrc and .prettierrc. Check your code against these standards by running:
npm run lint
Beside running the lint command you can use eslint and prettier extension on vscode. Then with the predefined vscode config, it will automatically format your code on save.
Make sure there are no violation of the coding standards before requesting a pull request.
To learn more about used dependencies take a look at the following resources: