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

chore: wrap/split long lines #281

Merged
merged 3 commits into from
Aug 29, 2024
Merged

Conversation

ptoscano
Copy link
Contributor

  • wrap/split few lines (code & comments) longer than 88
  • drop no more needed per-file ignores from flake8 config

No behaviour changes, only formatting.

Wrap lines with comments longer than 88 characters to fit within 80
columns, like the majority of the lines with comments.

Signed-off-by: Pino Toscano <ptoscano@redhat.com>
Split a string that make a line longer than 88 characters into two
strings, so it fits into the configured line length.

No behaviour changes.

Signed-off-by: Pino Toscano <ptoscano@redhat.com>
All the issues that were ignored in those files were fixed, so drop the
ignores set in the configuration file.

Signed-off-by: Pino Toscano <ptoscano@redhat.com>
Copy link
Contributor

@m-horky m-horky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@m-horky m-horky merged commit f60e50b into RedHatInsights:master Aug 29, 2024
23 of 24 checks passed
@ptoscano ptoscano deleted the flake8-fixes branch August 29, 2024 08:09
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