-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ail 162/feature/setup precommit basic linting #95
Ail 162/feature/setup precommit basic linting #95
Conversation
3255434
to
1377e1d
Compare
Note for reviewers
All other files were modified according to the pre-commit hook rules (basically removal of trailing white spaces, additional of blank line at the bottom of every file and fixing typos in comments). These changes do not affect any logic or behaviour. |
3ad6bd6
to
35324c1
Compare
35324c1
to
2c03fbd
Compare
2c03fbd
to
e45214c
Compare
b20de66
to
facfdb2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👏 praise: Looking good! Nice to no longer have whitespace issues 🚀
Some very minor comments and then ready to approve
Problem Statement
Add "easy win" pre-commit rules to help to catch simple errors and reduce whitespace differences in PRs as well.
PR Actions - What was done
How was this change tested
Notes:
N/A