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

Update astroid to 2.8.2 #5126

Merged
merged 3 commits into from
Oct 7, 2021
Merged

Update astroid to 2.8.2 #5126

merged 3 commits into from
Oct 7, 2021

Conversation

cdce8p
Copy link
Member

@cdce8p cdce8p commented Oct 6, 2021

@cdce8p cdce8p added the dependency Label for github dependabot label Oct 6, 2021
@cdce8p cdce8p added this to the 2.12.0 milestone Oct 6, 2021
@cdce8p cdce8p marked this pull request as draft October 6, 2021 23:22
@cdce8p cdce8p added the Blocked 🚧 Blocked by a particular issue label Oct 6, 2021
@cdce8p
Copy link
Member Author

cdce8p commented Oct 6, 2021

Blocked by: pylint-dev/astroid#1199 (comment)

setup.cfg Outdated Show resolved Hide resolved
@Pierre-Sassoulas Pierre-Sassoulas marked this pull request as ready for review October 7, 2021 11:40
@Pierre-Sassoulas Pierre-Sassoulas removed the Blocked 🚧 Blocked by a particular issue label Oct 7, 2021
@cdce8p
Copy link
Member Author

cdce8p commented Oct 7, 2021

@Pierre-Sassoulas I would have updated it in a few minutes 😄
If you like, I can take care of updating / merging all requires astroid update PRs afterwards.

ChangeLog Outdated Show resolved Hide resolved
@Pierre-Sassoulas
Copy link
Member

If you like, I can take care of updating / merging all requires astroid update PRs afterwards.

I would appreciate that, thanks 😄 !

@cdce8p
Copy link
Member Author

cdce8p commented Oct 7, 2021

Before we merge it, did you plan on releasing astroid 2.8.3 with just the wrapt update or would you like to wait a bit longer?

@Pierre-Sassoulas
Copy link
Member

I can't right now but that would make sense. I can do that in ~= 6 hours . If you want you can try the release process ;)

@coveralls
Copy link

coveralls commented Oct 7, 2021

Pull Request Test Coverage Report for Build 1316008283

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 109 unchanged lines in 10 files lost coverage.
  • Overall coverage increased (+0.05%) to 93.179%

Files with Coverage Reduction New Missed Lines %
pylint/checkers/design_analysis.py 2 98.85%
pylint/checkers/init.py 2 93.94%
pylint/lint/report_functions.py 4 90.7%
pylint/message/message_handler_mix_in.py 9 95.85%
pylint/checkers/base_checker.py 10 85.57%
pylint/checkers/imports.py 10 94.64%
pylint/config/init.py 11 71.23%
pylint/checkers/similar.py 12 96.34%
pylint/lint/pylinter.py 20 92.57%
pylint/checkers/base.py 29 96.91%
Totals Coverage Status
Change from base Build 1313959962: 0.05%
Covered Lines: 13537
Relevant Lines: 14528

💛 - Coveralls

@cdce8p
Copy link
Member Author

cdce8p commented Oct 7, 2021

I can't right now but that would make sense. I can do that in ~= 6 hours . If you want you can try the release process ;)

I'll leave that to you (for now at least) 😅 Just wanted to know if I should wait before merging this one. No need to hurry for the release then. This won't be the last time we update the version before pylint 2.12 anyway.

@cdce8p cdce8p changed the title Update astroid to 2.8.1 Update astroid to 2.8.2 Oct 7, 2021
@cdce8p cdce8p merged commit c4b5441 into pylint-dev:main Oct 7, 2021
@cdce8p cdce8p deleted the astroid-2.8.1 branch October 7, 2021 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency Label for github dependabot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants