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

lint: Extend list of ignored flake8 errors, instead of overwriting it #608

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

Bjwebb
Copy link
Contributor

@Bjwebb Bjwebb commented Aug 7, 2024

@chrisarridge I've decided to just make the setup.cfg change for now, which ignores the + at start/end of lines entirely, as we'll look at fixing linting automatically with black later.

Copy link
Contributor

@chrisarridge chrisarridge left a comment

Choose a reason for hiding this comment

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

That's a good pragmatic choice and makes sense to me.

@Bjwebb Bjwebb merged commit 6a876b1 into merge-codeforiati-and-publishingstats Aug 7, 2024
2 checks passed
@Bjwebb Bjwebb deleted the fix-lint branch August 7, 2024 14: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