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

achhina

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.

Issue: Add command-line functionality to uuid module #88597

achhina

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

gpshead

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

pythonGH-88597 : Rename uuid cli args and parameters to be in line wit… …

bad954e

…h uuidgen.

bedevere-bot added the awaiting review label Jan 22, 2023

bedevere-bot mentioned this pull request Jan 22, 2023

Add command-line functionality to uuid module #88597

Closed

achhina commented Jan 22, 2023

View reviewed changes

Comment thread Doc/library/uuid.rst Show resolved Hide resolved

updated a missed -n to -N change in the docs.

f271cc6

gpshead approved these changes Jan 23, 2023

View reviewed changes

bedevere-bot added awaiting merge and removed awaiting review labels Jan 23, 2023

gpshead requested a review from serhiy-storchaka January 23, 2023 00:04

gpshead self-assigned this Jan 23, 2023

serhiy-storchaka approved these changes Jan 23, 2023

View reviewed changes

gpshead added the skip news label 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

bedevere-bot removed the awaiting merge label Jan 25, 2023

iritkatriel pushed a commit to iritkatriel/cpython that referenced this pull request Jan 25, 2023

pythonGH-88597 : Rename uuid's new CLI args to be in line with uuidgen. ( …

97081c9

python#101248) this way they match an existing uuidgen command line tool.

mdboom pushed a commit to mdboom/cpython that referenced this pull request Jan 31, 2023

pythonGH-88597 : Rename uuid's new CLI args to be in line with uuidgen. ( …

bf22554

python#101248) this way they match an existing uuidgen command line tool.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

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