FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

GH-88597: Rename uuid's new CLI args to be in line with uuidgen. by achhina · Pull Request #101248 · python/cpython · GitHub

Repository navigation

GH-88597: Rename uuid's new CLI args to be in line with uuidgen. - #101248

Merged
gpshead merged 2 commits into
python:mainfrom
achhina:GH-88597-1
Jan 25, 2023
Merged

gpshead merged 2 commits into
python:mainfrom
achhina:GH-88597-1

Conversation

achhina commented Jan 22, 2023 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

Follow-up patch in regards to the proposed changes mentioned here.

❯ ./python.exe -m uuid -h          
usage: uuid.py [-h] [-u {uuid1,uuid3,uuid4,uuid5}] [-n NAMESPACE] [-N NAME]

Generates a uuid using the selected uuid function.

options:
  -h, --help            show this help message and exit
  -u {uuid1,uuid3,uuid4,uuid5}, --uuid {uuid1,uuid3,uuid4,uuid5}
                        The function to use to generate the uuid. By default uuid4 function is used.
  -n NAMESPACE, --namespace NAMESPACE
                        The namespace is a UUID, or '@ns' where 'ns' is a well-known predefined UUID addressed by namespace name. Such as @dns,
                        @url, @oid, and @x500. Only required for uuid3/uuid5 functions.
  -N NAME, --name NAME  The name used as part of generating the uuid. Only required for uuid3/uuid5 functions.

Comment thread Doc/library/uuid.rst

achhina commented Jan 22, 2023

Copy link
Copy Markdown
Contributor Author

Hi @gpshead, would you be able to take a look at this when you get a chance?

gpshead commented Jan 23, 2023

Copy link
Copy Markdown
Member

I think this looks good, I looped @serhiy-storchaka in to see if he's got any more suggestions.

gpshead self-assigned this Jan 23, 2023
gpshead changed the title GH-88597: Rename uuid cli args and parameters to be in line with uuidgen. GH-88597: Rename uuid's new CLI args to be in line with uuidgen. Jan 23, 2023
gpshead merged commit 952a1d9 into python:main Jan 25, 2023
iritkatriel pushed a commit to iritkatriel/cpython that referenced this pull request Jan 25, 2023
mdboom pushed a commit to mdboom/cpython that referenced this pull request Jan 31, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL