-
-
Notifications
You must be signed in to change notification settings - Fork 412
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
Supporting .pypirc
in publish command
#792
Comments
pypirc
for in publish command.pypirc
for in publish command
.pypirc
for in publish command.pypirc
in publish command
Just found this crate from Charlie Marsh, it is probably what is needed to parse the .pypirc. I tried a little bit with expecting it to be a .toml and that didn't work out at all. I'm pretty sure a .INI-style is used. https://github.com/charliermarsh/configparser-rs I just figured I could share this here for future notice. |
Thanks @cnpryer for pointing this out. Originally posted by @camilo-s in #759 (comment)
|
@camilo-s just FYI, it is possible to supply them via command line in the current version or Rye. It should be possible to either get this information from Azure or setting it up locally on your machine and look at your local .pypirc. Everything is stored in clear text so it is just copy/pasting. I.e. you wouldn't have to run the task TwineAuthenticated-task at all. I have this in my Github Action when publishing to a private repo on Google Cloud: rye publish -r ${{ env.REPO_NAME }} --repository-url ${{ env.REPO_URL }} --username ${{ env.USERNAME }} --token ${{ env.PASSWORD }} -y And that works great for me. |
Assuming #759 is merged, it'd be nice to add
--config-file
dispatch to more consciously support.pypirc
(I can't remember if Twine will dynamically load this with incomplete arguments).It might be nice to implement this so that
.pypirc
can be used in place of Rye's credentials management.So something like this would tell Rye to disregard managed credentials in favor of the
.pypirc
file.And if this pattern works well, we could extend it to support other credentials files.
Originally posted by @cnpryer in #759 (comment)
The text was updated successfully, but these errors were encountered: