Skip to content

Pre-commit code clean up #37

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.

Sign up for GitHub

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

Jump to bottom
Merged
JustinPorter88 merged 13 commits into from
Sep 12, 2025
Merged

Pre-commit code clean up #37

JustinPorter88 merged 13 commits into from
Sep 12, 2025

Conversation

@JustinPorter88
Copy link
Contributor

@JustinPorter88 JustinPorter88 commented Sep 11, 2025

  1. Added pre-commit to the code base.
  2. Ran linting. 
    pre-commit run --all-files
  3. Resolved errors raised.
    image
  4. Ran dos2unix on all files.

@JustinPorter88
Adding pre-commit details. Have not run on full repo yet.
8779b2d
@JustinPorter88
Merge branch 'develop' of github.com:WISDEM/SONATA into precommit
8f498d2
@JustinPorter88
Ran pre-commit and did auto updates, but bypassing code conventions, …
9524db9 
…so still needs cleanup.
@JustinPorter88
Clean up for code convention of classCBM.py
1775daa
@JustinPorter88
Cleanup in cbm files to match code conventions enforced by precommit.
8d8dea6
@JustinPorter88
More code convention cleanup, and trying to have try/except/raise be …
284cb80 
…correct.
@JustinPorter88
Clean up of mesh files for code standards.
bd11e83
@JustinPorter88
More mesh and layer file cleanup for code standards.
21e7666
@JustinPorter88
Code standards clean ups on several files. Only a few more things lef…
b088bc9 
…t to clean up.
@JustinPorter88
Finishing clean up for pre-commit. All pre-commit checks now pass.
f10f193
@JustinPorter88
Ran dos2unix on everything as 'find . -type f -exec dos2unix {} \;' t…
daaa946 
…o have consistent endline characters.
@JustinPorter88
Adding blame ignores for linting commits.
60078b6
@JustinPorter88
Adding pre commit checks to github CI.
2f250f7
@JustinPorter88 JustinPorter88 marked this pull request as ready for review September 12, 2025 14:18
@JustinPorter88 JustinPorter88 merged commit 20073ea into develop Sep 12, 2025
28 of 39 checks passed
@JustinPorter88 JustinPorter88 deleted the precommit branch September 12, 2025 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JustinPorter88