-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feat 2 pyproject toml and linter #102
Conversation
@stellaprins This is useful for applying consistent formatting to the python scripts. The formatting shouldn't really be applied to auto generated XML or python files though, or files taken verbatim from external resources. We can discuss in the meeting though. |
I think tyhe main ones to ignore are in the Brian and NEURON folders. Basically anything you or Rob haven't added/updated should probably be left as is. |
@stellaprins Looks like there's been over 5000 new files in a recent commit. Assume that wasn't intended. Wouldn't be good to get this integrated with development/main. It may even be best to remove the offending commit from the commit history..? |
@pgleeson I'm working out whether I can fit the repo into the UCL ARC recommended template for Python projects using cookiecutter and probably have accidentily commited some local files that should have been ignored by git! Yep, won't merge these into dev. I'll undo the commit (or start a fresh branch and remove this one). |
Ok, great. Glad to see a nice comprehensive template for a full Python project at ARC. Note not everything there is probably required as this repo is not intended as a pip installable package, just whatever will make a cleaner simpler set of files. |
No description provided.