Add pre-commit workflow

[jacubsmith]

Description Adds lint check before a commit is allowed to be submitted.

Fixes # (issue reference) closes #57

Type of change

• [ ] Bug fix (non-breaking change which fixes an issue)
• [x] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] This change requires a documentation update

Checklist

• [x] I have read the Contribution Guidelines
• [x] My code follows the style guidelines of this project
• [x] I have performed a self-review of my own code
• [x] I have commented my code, particularly in hard-to-understand areas
• [x] My changes generate no new warnings
• [x] Any dependent changes have been merged and published in downstream module

Expected Outcome Linting checks will now be run whenever making a commit

[ayushagg31]

So basically whenever someone make changes and commits, pre-commit will run lint:all and if it fails it will remove those changes from staging. Is that the idea?

[jacubsmith]

Not quite, before it commits any changes it will run the linter and if it fails you can either fix the linting issues or explicitly commit it with the errors

[ayushagg31]

cool, it will give us more control over code quality, thanks.