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

Changes due to updated black #432

Merged
merged 2 commits into from Oct 19, 2023

Conversation

baperry2
Copy link
Contributor

Black released a new version with somewhat different formatting. In particular, the number of blank lines after the docstring at the top of files is standardized (perhaps this change: psf/black#3932). This causes flake8 to fail with E302 on files with no imports - black wants there to be one blank line, flake8 wants two. Therefore, it is now necessary to ignore flake8 E302.

@jrood-nrel jrood-nrel merged commit ebcb668 into AMReX-Combustion:development Oct 19, 2023
8 checks passed
@baperry2 baperry2 deleted the black-flake8 branch October 19, 2023 15:37
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.

None yet

2 participants