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

fix(manager/uv): skip unsupported sources #31270

Merged

Conversation

mkniewallner
Copy link
Contributor

@mkniewallner mkniewallner commented Sep 8, 2024

Changes

Skip dependencies that define a source that is not yet supported by Renovate.

Context

This emerges from discussion in #31186 (comment). This will avoid having Renovate try to search for dependencies in PyPI if the dependencies are actually mapped to other sources (like a git repository, or a local path).

The end result is really similar to what is done in Cargo, and is defensive against potential new sources that could be added.

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@mkniewallner mkniewallner force-pushed the fix/skip-non-pypi-dependencies-uv branch 2 times, most recently from 591bcb0 to 7183564 Compare September 8, 2024 22:22
@mkniewallner mkniewallner changed the title fix(manager/uv): skip non-PyPI dependencies fix(manager/uv): skip unsupported sources Sep 8, 2024
@mkniewallner mkniewallner force-pushed the fix/skip-non-pypi-dependencies-uv branch from 7183564 to 0eba47d Compare September 8, 2024 22:33
@mkniewallner mkniewallner marked this pull request as ready for review September 8, 2024 22:47
@mkniewallner mkniewallner requested a review from rarkins September 9, 2024 07:07
rarkins
rarkins previously approved these changes Sep 9, 2024
@rarkins rarkins requested review from viceice and secustor September 9, 2024 07:51
@mkniewallner mkniewallner requested a review from viceice September 9, 2024 16:44
@rarkins rarkins added this pull request to the merge queue Sep 9, 2024
Merged via the queue into renovatebot:main with commit e5eab53 Sep 9, 2024
38 checks passed
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 38.73.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mkniewallner mkniewallner deleted the fix/skip-non-pypi-dependencies-uv branch September 9, 2024 22:02
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 12, 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.

5 participants