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

solved mypy errors for poetry.mixology #5402

Merged

Conversation

tarun-jethwani
Copy link
Contributor

@tarun-jethwani tarun-jethwani commented Apr 3, 2022

Pull Request Check List

Relates-to: #5017

This PR fixes mypy errors for entire poetry.mixology package and remove those corresponding fixed modules from [[tool.mypy.overrides]] section in pyproject.toml file

src/poetry/mixology/failure.py Outdated Show resolved Hide resolved
src/poetry/mixology/incompatibility.py Show resolved Hide resolved
src/poetry/mixology/partial_solution.py Outdated Show resolved Hide resolved
@tarun-jethwani tarun-jethwani force-pushed the mypy-fix-errors-for-poetry-mixology branch from 45c72de to 6c2a5ed Compare April 15, 2022 08:30
@tarun-jethwani
Copy link
Contributor Author

resolved merge conflicts

Copy link
Member

@abn abn left a comment

Choose a reason for hiding this comment

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

Good work @tarun-jethwani few minor suggestions/questions.

src/poetry/mixology/failure.py Outdated Show resolved Hide resolved
src/poetry/mixology/incompatibility.py Show resolved Hide resolved
src/poetry/mixology/incompatibility.py Outdated Show resolved Hide resolved
src/poetry/mixology/incompatibility.py Show resolved Hide resolved
src/poetry/mixology/incompatibility.py Outdated Show resolved Hide resolved
@tarun-jethwani
Copy link
Contributor Author

Hi @abn , thanks for your comments and review, I have resolved and taken care of your suggestions and questions,
please verify the requested changes

Copy link
Member

@branchvincent branchvincent left a comment

Choose a reason for hiding this comment

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

thanks @tarun-jethwani, this looks great to me. I'll wait a bit before merging to see if @abn has any other comments

@branchvincent branchvincent requested a review from abn April 27, 2022 12:55
@abn
Copy link
Member

abn commented Apr 27, 2022

Thanks for handling this @branchvincent. @tarun-jethwani thank you for your patience, I think this looks good - will merge it shortly.

@abn abn merged commit 1abf363 into python-poetry:master Apr 27, 2022
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 29, 2024
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.

None yet

4 participants