Skip to content

Don't consider Union to always be a type alias#8489

Merged
DanielNoord merged 2 commits into
pylint-dev:mainfrom
DanielNoord:typevar-name
Mar 23, 2023
Merged

Don't consider Union to always be a type alias#8489
DanielNoord merged 2 commits into
pylint-dev:mainfrom
DanielNoord:typevar-name

Conversation

@DanielNoord

Copy link
Copy Markdown
Collaborator

Type of Changes

Type
🐛 Bug fix

Description

Closes #8487.

Silly mistake in this logic.

@DanielNoord DanielNoord added False Positive 🦟 A message is emitted but nothing is wrong with the code backport maintenance/4.0.x labels Mar 23, 2023
@DanielNoord DanielNoord added this to the 2.17.2 milestone Mar 23, 2023
@codecov

codecov Bot commented Mar 23, 2023

Copy link
Copy Markdown

Codecov Report

Merging #8489 (e53297a) into main (3570860) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #8489   +/-   ##
=======================================
  Coverage   95.81%   95.81%           
=======================================
  Files         174      174           
  Lines       18333    18334    +1     
=======================================
+ Hits        17566    17567    +1     
  Misses        767      767           
Impacted Files Coverage Δ
pylint/checkers/base/name_checker/checker.py 98.58% <100.00%> (+<0.01%) ⬆️

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM !

Comment thread pylint/checkers/base/name_checker/checker.py Outdated
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
@DanielNoord DanielNoord enabled auto-merge (squash) March 23, 2023 21:33
@DanielNoord DanielNoord merged commit 07127ee into pylint-dev:main Mar 23, 2023
github-actions Bot pushed a commit that referenced this pull request Mar 23, 2023
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
(cherry picked from commit 07127ee)
@github-actions

Copy link
Copy Markdown
Contributor

🤖 According to the primer, this change has no effect on the checked open source code. 🤖🎉

This comment was generated for commit e53297a

DanielNoord added a commit that referenced this pull request Mar 23, 2023
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
(cherry picked from commit 07127ee)
@DanielNoord DanielNoord deleted the typevar-name branch April 20, 2026 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backported False Positive 🦟 A message is emitted but nothing is wrong with the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

invalid-name type alias false-positive for union values

2 participants