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

Describe that repository-url parameter may be used to pass .pypirc repository name #985

Closed
pwlodarczyk92 opened this issue Jun 25, 2022 · 1 comment · Fixed by #987
Closed
Labels
enhancement New feature or request

Comments

@pwlodarczyk92
Copy link

It took me quite some time to find out how to publish a package to a private repository with credentials passed via .pypirc file. The most obvious thing was to pass the repository url via command line parameter, but the script was still asking me for a username and a password. I needed to take a look at the sources to find out that this parameter can be also used to pass repository name, and only then the .pypirc file is used.

This behaviour was a bit difficult for me to discover because:

  • parameter name and description suggest that there's nothing else than a url that can be passed via this parameter
  • in case of upload.pypi.org registry url, the registry name is inferred automatically and .pypirc file is used as intended
  • the .pypirc file was configured automatically by the aws codeartifact cli, so I didn't even know that there is a repository name distinct from the url that I can use to identify it.

Thanks for working on this package!

@pwlodarczyk92 pwlodarczyk92 added the enhancement New feature or request label Jun 25, 2022
@messense
Copy link
Member

I think we should follow twine and split the current repository-url option to repository and repository-url.

https://twine.readthedocs.io/en/stable/#twine-upload

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants