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

docs(pip-compile): Add note re proper usage of index-url #29421

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

craigds
Copy link
Contributor

@craigds craigds commented Jun 4, 2024

Changes

Renovate doesn't parse the --index-url option from the lockfiles, it just runs pip-compile and pip-compile finds its own --index-url (from the .in files instead.)

The docs actually already do state which file the --index-url should go in, but since some of the other config comes from the lockfile, I didn't expect to have to add the index URL in the .in file and so I glossed over that.

This PR draws attention to that detail so other people won't gloss over it :)

Context

borne from my confusion here.

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

rarkins
rarkins previously approved these changes Jun 4, 2024
@rarkins rarkins enabled auto-merge June 4, 2024 04:07
@rarkins rarkins changed the title pip-compile: Add note re proper usage of index-url docs(pip-compile): Add note re proper usage of index-url Jun 4, 2024
Renovate doesn't parse the `--index-url` option from the lockfiles, it
just runs pip-compile and pip-compile finds its own `--index-url` (from
the `.in` files instead.)

This change adds a note to the docs to draw attention to that
auto-merge was automatically disabled June 4, 2024 20:22

Head branch was pushed to by a user without write access

@craigds craigds force-pushed the pip-compile-index-url-note branch from 4a889b1 to 041f8f3 Compare June 4, 2024 20:22
@rarkins rarkins enabled auto-merge June 5, 2024 05:32
@rarkins rarkins added this pull request to the merge queue Jun 5, 2024
Merged via the queue into renovatebot:main with commit da580ad Jun 5, 2024
39 of 40 checks passed
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 37.391.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

kosmoz pushed a commit to kosmoz/renovate that referenced this pull request Jun 12, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 6, 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.

3 participants