Skip to content

SONARPY-819 Update Typeshed to support Python 3.9#881

Merged
pynicolas merged 2 commits intomasterfrom
SONARPY-819c
Jan 26, 2021
Merged

SONARPY-819 Update Typeshed to support Python 3.9#881
pynicolas merged 2 commits intomasterfrom
SONARPY-819c

Conversation

@andrea-guarino-sonarsource
Copy link
Copy Markdown
Contributor

@andrea-guarino-sonarsource andrea-guarino-sonarsource commented Jan 26, 2021

Typeshed maintenairs decided to split 2and3/builtins.pyi into 2/builtins.pyi and 3/builtins.pyi.
This change has quite a few consequences on sonar-python, because we don't currently have a simple way to merge python2 and python3 symbols with the same fully qualified name (we rely instead on creating a wrapper symbol, named AmbigousSymbol).

In order to avoid having many AmbigousSymbols for builtins (like str or int), I decided to copy latest version of 2and3/builtins.pyi and let sonar-python rely on it.

I created SONARPY-830 to remove this copied version and use instead latest 2/builtins.pyi and 3/builtins.pyi.

@sonarsource-next
Copy link
Copy Markdown

Kudos, SonarQube Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@pynicolas pynicolas merged commit 9b64b93 into master Jan 26, 2021
@pynicolas pynicolas deleted the SONARPY-819c branch January 26, 2021 15:26
hashicorp-vault-sonar-prod Bot pushed a commit that referenced this pull request Feb 23, 2026
GitOrigin-RevId: 7248309c6bfbfc1f6f7995e2dcdeac65d622a484
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants