-
Notifications
You must be signed in to change notification settings - Fork 310
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
Mention that twine prompts you for credentials when omitted #118
Comments
You also could have sent a PR with that information 😄 Feel free to do so anyway. We're all counting on you. 😛 |
It's been a while since I opened the issue and I do not remember exactly why I didn't send a PR, but I assume that, as I said, Also, I really like this feature, but there could be someone that says that you are not following the official specification of that file, where you must have username and password (
I will do that then :) By the way, what are your plans for #11? Is it okay if I added the current documentation from the README to the docs directory so that we start building a real one for https://twine.readthedocs.org? |
Please don't do that in the same pull request. If you want to work on #11 that should be a separate pull request. Given the timeline for Warehouse, I don't think twine will be around much longer. |
Absolutely.
Oh, got it. |
It's June. Should we mention this in the docs? While we're waiting for Warehouse and Twine/Pip integration? |
Sorry, I totally forgot about this. Is there a schedule for the integration? |
If you make changes in this regard, please consider also mentioning the feature added in #208 where it won't prompt for credentials if it finds them in a (secure) keyring. |
I'm a newish Twine maintainer. Thanks for the suggestion for improvement! A status update: Warehouse, the codebase behind the new PyPI, is available as a pre-production site at https://pypi.org, we're making steady progress on the developer roadmap thanks to funding from Mozilla's Open Source Support Program, and it's on its way to replacing the legacy PyPI site this year. We're currently seeking feedback from package maintainers, including via several IRC livechats/office hours this week, about what does or doesn't work for you in the new interface, and in the next few weeks we'll be doing so with the wider Python developer community. I'm working on releasing Twine 1.10.0 in the next few days. In #154 people are discussing the long-term vision for Twine, which is definitely up in the air, but clearly Twine has stuck around for several years and it's not about to disappear in the next few weeks or months. We've improved Twine user (and developer) documentation a lot in the past couple months -- see http://twine.readthedocs.io/ (@felipedau, as you suggested, I included the I think documenting Twine's behavior and |
@felipedau with #323 merged, I think we have a good template for documentation for this topic. Would you be interested in submitting a PR? |
Hi @brainwane! I'm really sorry for taking such a long time to get back to you. I hope that #340 is close to what you had in mind. Thank you very much for your work and everyone's who is also involved in this project backed by Mozilla Open Source Program. It's a huge benefit to the Python community! |
The docs say "twine will automatically prompt you for username or password". That seems satisfactory for now. Thanks felipedau for the attempted PR on this issue, but I think the system is now WAI. Feel free to re-engage the effort if you find the behavior is still surprising. |
Suppose you have something like this in
~/.pypirc
:I am not sure if this is an "intended feature" because if you call the usual
python setup.py upload
withusername
andpassword
omitted like that, you get an error:However, when calling
twine upload
it conveniently asks you for the credentials!I just opened this issue because I think this feature should be mentioned on the README (the only documentation I could find) for people that do not like to store plaintext passwords on disk.
Thanks!
The text was updated successfully, but these errors were encountered: