You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, we have considerably a large number of files being altered and moved within the repository. I am recommending the addition of pre-commit configurations to setup all the contributions to the same standard we would like to have in the project.
Implementation considerations
The pre-commit configurations could possibly validate the following:
Dockerfile linting
Shell formatting and linting
Missing trailing whitespaces
Markdown linting
End of file Fixer
Check for large files
Check Merge conflicts
Check yaml
Check xml
The list is based on the current status of the repository. This could also potentially allow us to extend the CI for pre-commit validation as a part of the workflow.
The text was updated successfully, but these errors were encountered:
Related to space-ros#192
Add precommit configs for
- file testing
- docker linting
- shell linting and formatting
- python linting and formatting
- cpp linting and formatting
Feature description
Currently, we have considerably a large number of files being altered and moved within the repository. I am recommending the addition of pre-commit configurations to setup all the contributions to the same standard we would like to have in the project.
Implementation considerations
The
pre-commit
configurations could possibly validate the following:The list is based on the current status of the repository. This could also potentially allow us to extend the CI for pre-commit validation as a part of the workflow.
The text was updated successfully, but these errors were encountered: