-
Notifications
You must be signed in to change notification settings - Fork 11
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
specify party name is required #112
Conversation
Co-authored-by: A.J. Stein <[email protected]>
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.
So I talked with others on the team and they actually agreed that the short-name
is not a requirement.
Because the diff crosses a deleted line, I cannot put in a suggestion, so @wandmagic, can you edit this to remove the previous sentence that requires short-name
? Only name
is really necessary.
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.
I would add an "a" in the cleanup before merge, but thanks for taking care of this so quickly.
Co-authored-by: A.J. Stein <[email protected]>
* Update oscal-metadata.md * Update content/documentation/general-concepts/oscal-metadata.md Co-authored-by: A.J. Stein <[email protected]> * Update oscal-metadata.md * Update oscal-metadata.md Co-authored-by: A.J. Stein <[email protected]> --------- Co-authored-by: A.J. Stein <[email protected]>
specify that party name is required in the documentation