-
Notifications
You must be signed in to change notification settings - Fork 32
Open
Description
While doing the MR #31 I followed the CONTRIBUTING step-by-step and I have some feedbacks / remarks / problems:
- Once the conda environment is installed, the
pre-commitwas not available (even if the it is available inenvironment_dev.yaml). I tried to runpip install pre-commitbut it did not solve the issue. However, usingconda install -c conda-forge pre-commitsolved the issue. Maybe we can update theenvironment_dev.yamlby moving thepre-commitrequirement frompiptodependencies(+ addconda-forgechannel which is commented for now) - Once the
pre-commitis installed, it is advised to launchpre-commit run --all-files. With the latest version ofmainI had several errors, cf:

- Regarding the
pre-commit,isortis mentioned in theCONTRIBUTING.mdbut it is not defined in the.pre-commit-config.yaml(Remark:mypyis used but not mentioned in theCONTRIBUTING.md) - To ensure the git conventions you can add a hook in the pre-commit by adding the following lines in the .pre-commit-config.yaml`:
- repo: https://github.com/alessandrojcm/commitlint-pre-commit-hook
rev: v2.2.0
hooks:
- id: commitlint
stages: [commit-msg]
additional_dependencies: ["@commitlint/config-conventional"]Some information regarding my laptop:
- Pop OS: 19.10
- conda: 4.8.2
Metadata
Metadata
Assignees
Labels
No labels