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

Missing return for link.is_wheel==True #4966

Closed
2 tasks done
Anselmoo opened this issue Jan 1, 2022 · 3 comments · Fixed by #4967
Closed
2 tasks done

Missing return for link.is_wheel==True #4966

Anselmoo opened this issue Jan 1, 2022 · 3 comments · Fixed by #4967
Labels
kind/bug Something isn't working as expected

Comments

@Anselmoo
Copy link
Contributor

Anselmoo commented Jan 1, 2022

  • I am on the latest Poetry version.
  • I have searched the issues of this repo and believe that this is not a duplicate.
  • OS version and name: macOS:latest
  • Poetry version: 1.1.12
  • Link of a Gist with the contents of your pyproject.toml file:

Issue

According to PR #4962, there is the question left, if pass should not be replaced by a return. This would be also consistent with the typing definition Optional[Link].

def get_cached_archive_for_link(self, link: Link) -> Optional[Link]:
# If the archive is already a wheel, there is no need to cache it.
if link.is_wheel:
pass

At the moment

archives = self.get_cached_archives_for_link(link)

will be always excetuded.

@Anselmoo Anselmoo added kind/bug Something isn't working as expected status/triage This issue needs to be triaged labels Jan 1, 2022
Anselmoo added a commit to Anselmoo/poetry that referenced this issue Jan 1, 2022
@Anselmoo
Copy link
Contributor Author

Anselmoo commented Jan 3, 2022

@finswimmer issue is assigend

@finswimmer
Copy link
Member

Thanks @Anselmoo 👍 I've asked the other maintainers to have a look onto the PR, because I cannot estimate the impact of this change (Even if I guess it's correct).

neersighted pushed a commit that referenced this issue Jan 17, 2022
@abn abn removed the status/triage This issue needs to be triaged label Mar 3, 2022
Copy link

github-actions bot commented Mar 2, 2024

This issue 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 Mar 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/bug Something isn't working as expected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants