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

Formatting: unify newlines by Black’s standards #360

Merged
merged 1 commit into from
Jul 24, 2019

Conversation

Glutexo
Copy link
Collaborator

@Glutexo Glutexo commented Jul 19, 2019

Add newlines between class and function definitions and other pieces of code that need to be visually separated. Also remove excessive newlines. Used Black to fix these cases.

This is a part to fix formatting in the whole codebase to be able to run Black, Flake8 and other tools automatically. #335

Add newlines between class and function definitions and other pieces of
code that need to be visually separated. Also remove excessive newlines.
Used Black to fix these cases.
@Glutexo Glutexo self-assigned this Jul 19, 2019
@Glutexo Glutexo mentioned this pull request Jul 22, 2019
@Glutexo
Copy link
Collaborator Author

Glutexo commented Jul 24, 2019

This pull request has been approved. Can we merge it?

@dehort dehort merged commit 4d7d094 into RedHatInsights:master Jul 24, 2019
@Glutexo Glutexo deleted the black_newlines branch July 25, 2019 07:08
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.

3 participants