Skip to content
Snippets Groups Projects
CONTRIBUTING.md 3.95 KiB
Newer Older
# How to contribute to FAIDARE
Célia Michotey's avatar
Célia Michotey committed
[[_TOC_]]
Raphaël Flores's avatar
Raphaël Flores committed

Raphaël Flores's avatar
Raphaël Flores committed
## Git management

### Branches

Raphaël Flores's avatar
Raphaël Flores committed
* One stable & protected [master](/) branch
Raphaël Flores's avatar
Raphaël Flores committed
* Feature branches for development following the pattern `[dev_type]/[dev_name]` (ie. `chore/explaining_how_to_merge`) where `[dev_type]` can be:
  * fix (bug fixes)
  * feat (new feature)
  * style (style modification)
  * refactor (code refactoring)
  * chore (base maintenance such as version bump)
  * test (for anything related to tests)
Raphaël Flores's avatar
Raphaël Flores committed

### Commit & branches merge
Raphaël Flores's avatar
Raphaël Flores committed
* Commit name should follow pattern `[dev_type]: [brief description of the commit, lower than 50 characters]. [JIRA-KEY]`
* All branches must be merged via a `merge request` (MR)
* Merge requests should be created at the time of the branch creation in order to allow reviewer to comment and follow the developments, specify the `WIP` tag in the MR name (go further: *[feature highlight WIP](https://about.gitlab.com/2016/01/08/feature-highlight-wip/)*). Example:

  ```sh
  git checkout -b chore/explaining_how_to_merge
  git push --set-upstram origin chore/explaining_how_to_merge
  ```

  Returns a link for creating the merge request easily:

  ```sh
  Total 0 (delta 0), reused 0 (delta 0)
  remote:
  remote: To create a merge request for chore/explaining_how_to_merge, visit:
  remote:   https://forgemia.inra.fr/urgi-is/faidare/merge_requests/new?merge_request%5Bsource_branch%5D=chore/explaining_how_to_merge
  remote:
  To forgemia.inra.fr:urgi-is/faidare.git
  * [new branch]      chore/explaining_how_to_merge -> chore/explaining_how_to_merge
  La branche 'chore/explaining_how_to_merge' est paramétrée pour suivre la branche distante 'chore/explaining_how_to_merge' depuis 'origin'.
  ```
Raphaël Flores's avatar
Raphaël Flores committed

* A `git rebase` is strongly recommanded before merging a MR
  * [Git rebase official documentation](https://git-scm.com/book/en/v2/Git-Branching-Rebasing)
  * [How to keep a clean history](https://about.gitlab.com/2018/06/07/keeping-git-commit-history-clean/)
Raphaël Flores's avatar
Raphaël Flores committed
* Merge requests should be reviewed by at least 2 colleagues
* Continuous Integration is launched automatically by Gitlab on each commit push or merge request creation.

## Data management
Raphaël Flores's avatar
Raphaël Flores committed
* Git LFS (look at [how to install](https://git-lfs.github.com/)) feature is enabled on this project in order to store JSON test data
Raphaël Flores's avatar
Raphaël Flores committed
* Another dedicated Git LFS project (internal only) will be created to handle all private + public JSON files
Raphaël Flores's avatar
Raphaël Flores committed
* **TODO**: refer to the good Git LFS project in the CI settings for indexing the relevant data into the relevant Elasticsearch indices/instances
Raphaël Flores's avatar
Raphaël Flores committed
* the JSON files generation will be handled by an external ET (extract/transform) tool.

## Development environment
Raphaël Flores's avatar
Raphaël Flores committed
* Look at the [README.md](README.md) for installation and execution instructions.
* Recommanded IDE is [Intellij IDEA](https://www.jetbrains.com/idea/)
* Use linting to apply code standards within the team:
  * Use `yarn format` (for `web` code only)
  * Use [Checkstyle](https://checkstyle.org/) and [PMD](https://pmd.github.io/) (**TODO**: implement) for backend code
Raphaël Flores's avatar
Raphaël Flores committed
* All runtime variables should be externalized from the code in order to facilitate the CI management (database host/port, application name, public URL, JSON location...) and the adoption by partners
* Configure [Yelp's detect-secrets](https://github.com/Yelp/detect-secrets) pre-commit hook to track any high entropy strings, which could possibly be leaked secrets.
  * Run: `pip3 install pre-commit detect-secrets --user && pre-commit install` to configure you dev environment.
  * If a secret is detected:
    * either fix it (because it must not be committed)
    * or run:
      * `detect-secrets scan --update .secrets.baseline` to update the secret baseline, then
      * `detect-secrets audit .secrets.baseline` to tag it as a false positive if relevant.
Raphaël Flores's avatar
Raphaël Flores committed
## Testing recommendations
Raphaël Flores's avatar
Raphaël Flores committed
Behaviour driven development (upon [TDD](https://dannorth.net/2012/05/31/bdd-is-like-tdd-if/)) is recommended for all new developments.

More information on what is BDD: <https://dannorth.net/introducing-bdd/>