-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
DRAFT Code style automation PLEASE DO NOT MERGE #312
Conversation
This is mostly a placeholder.
@springfall2008 I think this is ready to go now - once you've approved this, I will do a PR on Also, this relies on that pre-commit.com/lite thing being set up on your repo. |
I'm taking this one out of draft @springfall2008 , but please don't merge this one. I'm trying to test out the standards checking running on your repo. |
@springfall2008 I'm merging this from the code-style branch in my repo to a branch of the same name in your repo, to test out the code style pipeline. This isn't putting it into main any more (I'll do that separately another time). |
This closes issue #313 .
I suggest we use the issue for discussing what standards etc. that we want, then use this PR to review the actual implementation, to try and keep it cleaner.