search

NREL / GreenHEART

Other
2 stars 9 forks source link

Add ruff as an automatic linter #50

Open johnjasa opened 6 days ago

johnjasa commented 6 days ago

Add ruff as an automatic linter

We should add a pre-commit hook for linting GreenHEART to streamline and simplify code coming from disparate sources.

Proposed solution

Ruff is fast, easy-to-setup, and well-accepted. I'll defer to @RHammond2 about best settings though it should be pretty straightforward.

bayc commented 2 days ago

Agreed! We setup pre-commit hooks in the electrolyzer repo (inspired by FLORIS) that we could pull from as well: https://github.com/NREL/electrolyzer