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

Fix Pyreverse optional annotation bug #9016

Merged
merged 3 commits into from
Sep 9, 2023

Conversation

nickdrozd
Copy link
Contributor

@nickdrozd nickdrozd commented Sep 7, 2023

Type of Changes

Type
βœ“ πŸ› Bug fix

Description

Closes #9014

@nickdrozd nickdrozd requested a review from DudeNr33 as a code owner September 7, 2023 05:29
@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Merging #9016 (433bd85) into main (77dcaad) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #9016   +/-   ##
=======================================
  Coverage   95.75%   95.75%           
=======================================
  Files         173      173           
  Lines       18648    18648           
=======================================
  Hits        17856    17856           
  Misses        792      792           
Files Changed Coverage Ξ”
pylint/pyreverse/utils.py 94.20% <100.00%> (ΓΈ)

@github-actions

This comment has been minimized.

Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

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

The pre-commit issue seems genuine.

@nickdrozd
Copy link
Contributor Author

The typing around that section is a little squirrelly (see #8266). Should be fixed now.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2023

πŸ€– According to the primer, this change has no effect on the checked open source code. πŸ€–πŸŽ‰

This comment was generated for commit 433bd85

Copy link
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

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

πŸ‘

Copy link
Collaborator

@DudeNr33 DudeNr33 left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@DudeNr33 DudeNr33 merged commit a21f5d3 into pylint-dev:main Sep 9, 2023
github-actions bot pushed a commit that referenced this pull request Sep 9, 2023
* Add alternative optional bug test cases

* Fix Pyreverse optional annotation bug

* Fix type error

(cherry picked from commit a21f5d3)
Pierre-Sassoulas pushed a commit that referenced this pull request Sep 9, 2023
* Add alternative optional bug test cases

* Fix Pyreverse optional annotation bug

* Fix type error

(cherry picked from commit a21f5d3)

Co-authored-by: Nick Drozd <nicholasdrozd@gmail.com>
@nickdrozd nickdrozd deleted the pyreverse-optional-bug branch October 7, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backported Bug πŸͺ² pyreverse Related to pyreverse component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Redundant Optional in Pyreverse
3 participants