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 the annotations in pip/_internal/cli #4

Closed
wants to merge 21 commits into from

Conversation

DiddiLeija
Copy link
Owner

This is the second part of my job made on pypa#10018, where I must complete all the annotations from pip/_internal/cli.

DiddiLeija and others added 21 commits June 14, 2021 07:53
This is the second part of my job made on pypa#10018, where I must complete all the annotations from `pip/_internal/cli`.
This is the news file for my pull request.
I just changed one function annotations, on line 121.
Just update the annotations according to the CI last failure.
I wanted to use "RequirementPreparer", not "RequirementParser".
There were some previous errors with this, but now I fixed them.
It is not exactly a problem, it's just to pass the CI.
They fail on the CI checks.
Convert type commentaries into annotations.
Fix also the original annotation instead of just copying it.

Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>
Some of the annotations were incorrect.
It is "self._session: Optional[PipSession] = None" instead of "self._session = None # type: Optional[PipSession]".
They were imported but unused.
One of them was incorrect.
Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>
@DiddiLeija DiddiLeija closed this Jun 29, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant