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 used-before-assignment FP for generic type syntax (Py 3.12) #9150

Merged

Conversation

jacobtylerwalls
Copy link
Member

Type of Changes

Type
βœ“ πŸ› Bug fix

Description

Closes #9110

@jacobtylerwalls jacobtylerwalls added False Positive 🦟 A message is emitted but nothing is wrong with the code backport maintenance/3.3.x labels Oct 15, 2023
@jacobtylerwalls jacobtylerwalls added this to the 3.0.2 milestone Oct 15, 2023
@jacobtylerwalls jacobtylerwalls added the C: used-before-assignment Issues related to 'used-before-assignment' check label Oct 15, 2023
@github-actions

This comment has been minimized.

@jacobtylerwalls jacobtylerwalls added the Needs astroid update Needs an astroid update (probably a release too) before being mergable label Oct 15, 2023
@jacobtylerwalls jacobtylerwalls added python 3.12 and removed Needs astroid update Needs an astroid update (probably a release too) before being mergable labels Oct 16, 2023
@codecov
Copy link

codecov bot commented Oct 16, 2023

Codecov Report

Merging #9150 (33f0dbe) into main (0796dfa) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #9150   +/-   ##
=======================================
  Coverage   95.76%   95.76%           
=======================================
  Files         173      173           
  Lines       18680    18682    +2     
=======================================
+ Hits        17889    17891    +2     
  Misses        791      791           
Files Coverage Ξ”
pylint/checkers/variables.py 97.23% <100.00%> (+<0.01%) ⬆️

@github-actions
Copy link
Contributor

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

This comment was generated for commit 33f0dbe

@Pierre-Sassoulas
Copy link
Member

Let's squash even if there's an astroid update and two commits so the backport is automatic ?

@jacobtylerwalls jacobtylerwalls merged commit cb29fbd into pylint-dev:main Oct 16, 2023
47 checks passed
@jacobtylerwalls jacobtylerwalls deleted the pep695-assign-same-line branch October 16, 2023 12:52
github-actions bot pushed a commit that referenced this pull request Oct 16, 2023
)

* Fix `used-before-assignment` FP for generic type syntax (Py 3.12)

* Bump astroid to 3.0.1

(cherry picked from commit cb29fbd)
Pierre-Sassoulas pushed a commit that referenced this pull request Oct 16, 2023
)

* Fix `used-before-assignment` FP for generic type syntax (Py 3.12)

* Bump astroid to 3.0.1

(cherry picked from commit cb29fbd)
Pierre-Sassoulas pushed a commit that referenced this pull request Oct 16, 2023
) (#9154)

* Fix `used-before-assignment` FP for generic type syntax (Py 3.12)

* Bump astroid to 3.0.1

(cherry picked from commit cb29fbd)

Co-authored-by: Jacob Walls <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backported C: used-before-assignment Issues related to 'used-before-assignment' check False Positive 🦟 A message is emitted but nothing is wrong with the code python 3.12
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect linting with type parameters PEP695 (generic)
3 participants