Skip to content

Conversation

@NoahGorny
Copy link
Contributor

This line has caused mypy to fail for me locally for the last few days..
Seems like mypy does not understand that sys.platlibdir will never be called unless this attribute exists

This does not warrant a news fragment IMO

@sbidoul sbidoul added the skip news Does not need a NEWS file entry (eg: trivial changes) label Sep 29, 2021
@sbidoul
Copy link
Member

sbidoul commented Sep 29, 2021

This triggers another error on python3.9 used by pre-commit in CI... it looks like @uranusjr is also addressing this in #10511. It's probably worth merging separately indeed because it is annoying.

@uranusjr
Copy link
Member

See https://github.com/pypa/pip/pull/10511/files#diff-dcf4783407c2fd65772c5e273a0e0554aa4273dc2242bb32bb41343d135a7498L67-R68

Feel free to copyt the same approach here since #10511 likely needs more time (for reviews).

@github-actions github-actions bot added the needs rebase or merge PR has conflicts with current master label Nov 22, 2021
@pradyunsg
Copy link
Member

Going to go ahead and close this, since it has merge conflicts that haven't been resolved in a while. Please feel welcome to file a new PR or to reopen and update this one!

@pradyunsg pradyunsg closed this Sep 30, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs rebase or merge PR has conflicts with current master skip news Does not need a NEWS file entry (eg: trivial changes)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants