oss-slu / Mouser

GNU General Public License v3.0
1 stars 7 forks source link

151 pass linting #173

Closed aobermiller closed 9 months ago

aobermiller commented 9 months ago

Fixes #151

What was changed?

Every python file was modified at least a little as well as the yaml file for the linting.

Why was it changed?

The yaml file was configured to test every file in the repository. The test was also configured to test for things we cared about for our linting. The yaml file also now makes sure to lint every file, even if a test fails. That way you can see what needs to change in every file.

Every file was modified to pass the linting test that I configured.

How was it changed?

Python files had unnecessary white space removed, and all modules, classes, and functions were doc-stringed. Many python files need their code changed to pass linting, but only syntactically. Actual functionality was kept the same.

Now, if one file fails the linting, then it fails at the very end after all files were checked instead of immediately.

loganwyas commented 9 months ago

@aobermiller There was a recent merge to main, would you be able to quickly fix the merge conflict?

aobermiller commented 9 months ago

@loganwyas merge conflict fixed.