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

Complete type annotations in pip/_internal/metadata #10168

Closed

Conversation

harupy
Copy link
Contributor

@harupy harupy commented Jul 17, 2021

@harupy harupy changed the title Complete annotations: pip/_internal/metadata Complete type annotations in pip/_internal/metadata Jul 17, 2021
@harupy
Copy link
Contributor Author

harupy commented Jul 17, 2021

@uranusjr @pradyunsg Could you review this PR?

@uranusjr
Copy link
Member

I'm going to delay this a bit since merging this first would cause conflicts in #10157 (it's easier to rebase this onto #10157).

@uranusjr
Copy link
Member

Conflicts as expected. Could you help resolve these? (I just tried resolving conflicts on GitHub and it seems to break CI triggers 😞 on other PRs)

@harupy
Copy link
Contributor Author

harupy commented Jul 23, 2021

@uranusjr Can I close this PR because iter_dependencies already has type annotation on the main branch?

@uranusjr
Copy link
Member

Oh! Maybe I did that as a part of the pkg_resources rewriting PR and didn't notice. Thanks!

@uranusjr uranusjr closed this Jul 23, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants