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.

Sign up for GitHub

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

Jump to bottom

Flake8 the code. #193

Merged
merged 1 commit into from
Apr 3, 2024
Merged

Flake8 the code. #193

merged 1 commit into from
Apr 3, 2024

Conversation

icemac
Copy link
Member

@icemac icemac commented Mar 28, 2024

After fixing isort it got visible that we did not do this, yet.

@icemac
Flake8 the code.
510895a 
After fixing isort it got visible that we did not do this, yet.
@icemac icemac requested a review from dataflake March 28, 2024 08:52
@icemac icemac self-assigned this Mar 28, 2024
dataflake
dataflake approved these changes Mar 28, 2024
View reviewed changes
Copy link
Member

@dataflake dataflake left a comment

Choose a reason for hiding this comment

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

Do you want to remove the remaining pylint comments as well?

@icemac
Copy link
Member Author

icemac commented Apr 3, 2024

Do you want to remove the remaining pylint comments as well?

We probably could do this, but I am too lazy to do it now.

Thank you for reviewing this PR. 😃

@icemac icemac merged commit 4b702c8 into master Apr 3, 2024
54 checks passed
@icemac icemac deleted the flake8 branch April 3, 2024 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dataflake dataflake dataflake approved these changes

Assignees

@icemac icemac

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@icemac @dataflake