Skip to content

Allow to specify source url in global pip config #2395

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

Closed
wants to merge 1 commit into from
Closed

Allow to specify source url in global pip config #2395

wants to merge 1 commit into from

Conversation

neo-sam
Copy link

@neo-sam neo-sam commented May 8, 2020

Pull Request Check List

Resolves: #1632 #559 #664

  • Added tests for changed code.
  • Updated documentation for changed code.

Let poetry resolve pip config global.index-url as default pypi repository if exists.

@neo-sam
Copy link
Author

neo-sam commented May 8, 2020

Auto merge review check bot fails because some old code in this file don't fit black format style.

@sdispater
Copy link
Member

Thanks for your contribution!

However, I do not feel comfortable using information coming from other tools automatically. This should be done explicitly by the user.

@sdispater sdispater closed this May 22, 2020
@neo-sam
Copy link
Author

neo-sam commented May 22, 2020

Hope there will be one way to specify custom default pip repository in poetry global config in the new version.

@DanielW10001
Copy link

@sdispater How about promoting the user to ask whether to use the global.index-url from pip config when it presents?

Copy link

github-actions bot commented Mar 1, 2024

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 Mar 1, 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