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

Use black on the entire project #64

Merged
merged 2 commits into from
Aug 23, 2022

Conversation

mcdonnnj
Copy link
Contributor

This pull request just expands the scope of black checking to cover the entire project. This ensures that any Python code in the project has the same formatting.

Have black check the entire project instead of only the scraper/
directory. This will ensure that all Python code in this project
is formatted consistently.
This ensures that the project passes the new test configuration.
@IanLee1521 IanLee1521 merged commit acaed67 into LLNL:main Aug 23, 2022
@IanLee1521
Copy link
Member

Thanks again @mcdonnnj !

I think this was done this way from back in the day when I would have a local venv/ directory that I didn't want black to mess with, and also some other scripts locally that weren't in the repo that I didn't want it to change... but going forward this is a better option.

@mcdonnnj mcdonnnj deleted the improvement/use_black_everywhere branch August 23, 2022 17:22
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.

2 participants