-
Notifications
You must be signed in to change notification settings - Fork 313
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
Add instructions on how to use keyring #323
Conversation
Codecov Report
@@ Coverage Diff @@
## master #323 +/- ##
=======================================
Coverage 69.53% 69.53%
=======================================
Files 12 12
Lines 581 581
Branches 91 91
=======================================
Hits 404 404
Misses 150 150
Partials 27 27 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great overall! A few small requests for improvement. Thank you!
README.rst
Outdated
.. code-block:: console | ||
|
||
$ keyring set https://upload.pypi.org/legacy/ your-username | ||
$ python3 -m keyring set https://upload.pypi.org/legacy/ your-username |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please include the "or" between these two lines as you did above.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
* ``TWINE_USERNAME`` - the username to use for authentication to the repository | ||
* ``TWINE_PASSWORD`` - the password to use for authentication to the repository | ||
* ``TWINE_USERNAME`` - the username to use for authentication to the repository. | ||
* ``TWINE_PASSWORD`` - the password to use for authentication to the repository. | ||
* ``TWINE_REPOSITORY`` - the repository configuration, either defined as a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For parallelism, could you add the ending period to all the items in this list?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
|
||
$ keyring set https://test.pypi.org/legacy/ your-username | ||
# or | ||
$ python3 -m keyring set https://test.pypi.org/legacy/ your-username |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you find some way here to indicate that it'll ask you to input a password? And I have the same suggestion for line 116-117.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
README.rst
Outdated
python3-dbus``. See `Keyring's installation instructions`_ for more details. | ||
|
||
Once keyring is installed you can use the ``keyring`` program to set your | ||
username and password for Twine. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick: I'd phrase this as:
"set your package index (repository) username and password for Twine to use"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call, I changed it even a bit more. LMK what you think.
Closes #277.
Helps pypa/packaging.python.org#297 and #11.