Skip to content
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

[repo meta] formatting & addition of pre-commit configuration #153

Closed
wants to merge 2 commits into from

Conversation

whi-tw
Copy link

@whi-tw whi-tw commented Sep 14, 2022

Disclaimer: This could be a massive over-step, so please feel free to tell me to stay in my lane.

Working on ercf.py in #150, I noticed that the file had quite a lot of formatting quirks, which I was constantly fighting with in my IDE.

I have reformatted the file with the latest version of black and also removed a lot of trailing whitespace.

Additionally I've added configuration for pre-commit which could potentially prevent things from getting messy again. I've only run it on ercf.py so far, hopefully if people use the pre-commit hook in future, everything will gradually be tidied up. pre-commit can be run locally (docs on the website above) and automatically for free with a CI check via https://pre-commit.ci/ (requires setup on @EtteGit's side).

@whi-tw whi-tw closed this Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant