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

Minor changes #1464

Merged
merged 2 commits into from
Mar 9, 2022
Merged

Minor changes #1464

merged 2 commits into from
Mar 9, 2022

Conversation

cdce8p
Copy link
Member

@cdce8p cdce8p commented Mar 9, 2022

Description

Spelling + unused type ignore

@cdce8p cdce8p added the Maintenance Discussion or action around maintaining astroid or the dev workflow label Mar 9, 2022
@cdce8p cdce8p added this to the 2.11.0 milestone Mar 9, 2022
@DanielNoord
Copy link
Collaborator

Do we want to turn on warn_unused_ignores?

@cdce8p
Copy link
Member Author

cdce8p commented Mar 9, 2022

Do we want to turn on warn_unused_ignores?

Someday 😅
These were just some things I had gathered over the last weeks.

@cdce8p cdce8p merged commit 81235ff into pylint-dev:main Mar 9, 2022
@cdce8p cdce8p deleted the misc branch March 9, 2022 21:20
@cdce8p
Copy link
Member Author

cdce8p commented Mar 9, 2022

Do we want to turn on warn_unused_ignores?

There is also the issue of python_version which should really be added add some point. Just to make runs with different versions reproducible. There just isn't any point in doing so before we've fixed all existing errors.

Something for another time. I need to take a step back from astroid / pylint soon to focus on other things, so please don't start with that now. Will have more time in a month or two.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintenance Discussion or action around maintaining astroid or the dev workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants