Skip to content
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 a parameters map to p:uuid? #587

Closed
ndw opened this issue Jul 29, 2024 · 1 comment
Closed

Add a parameters map to p:uuid? #587

ndw opened this issue Jul 29, 2024 · 1 comment

Comments

@ndw
Copy link
Collaborator

ndw commented Jul 29, 2024

It strikes me as slightly weird that the p:uuid step says:

Implementations must support version 4 UUIDs. Support for other versions of UUID, and the mechanism by which the necessary inputs are made available for computing other versions, is implementation-defined.

Wouldn't it make sense to add a parameters map option so that there's a place to provide the necessary inputs? Implementors could use extension attributes or something, I suppose, but it seems like there's plenty of precedent for having a parameters map for this purpose...

@xml-project
Copy link
Member

I thought that "the mechanism" refers to some type of configuration, or an extension attribute.
But yes, a parameters map is certainly a good way to do this.

@ndw ndw closed this as completed in 2d81781 Jul 30, 2024
ndw added a commit that referenced this issue Jul 30, 2024
Fix #587 by adding a parameters option
ndw added a commit to ndw/steps that referenced this issue Jul 31, 2024
ndw added a commit to ndw/steps that referenced this issue Jul 31, 2024
ndw added a commit to ndw/steps that referenced this issue Jul 31, 2024
ndw added a commit to ndw/steps that referenced this issue Aug 1, 2024
ndw added a commit to ndw/steps that referenced this issue Aug 1, 2024
ndw added a commit to ndw/steps that referenced this issue Aug 1, 2024
ndw added a commit to ndw/steps that referenced this issue Aug 1, 2024
ndw added a commit to ndw/steps that referenced this issue Aug 2, 2024
ndw added a commit to ndw/steps that referenced this issue Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants