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

Created metadata.yaml for reference to reusable elements #224

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

eddie-knight
Copy link
Contributor

No description provided.

@funnelfiasco
Copy link
Contributor

While this is still in draft status, I'd like to suggest using "LevelX" instead of "MX" for levels. It's more characters, but it also makes it more self-evident what it means. I'd accept "LX", but I'd be sad about it.

The Open Source Project Security (OSPS) Baseline is a set of security
criteria that projects should meet to demonstrate a strong security posture.
applicability-categories:
- id: Maturity Level 1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Constrain the identifiers to strings without spaces. These will get composed and fed into other systems that could may not handle multiple words

Suggested change
- id: Maturity Level 1
- id: MA-1

Signed-off-by: Eddie Knight <[email protected]>
Signed-off-by: Eddie Knight <[email protected]>
Copy link
Contributor

@SecurityCRob SecurityCRob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saving money by removing characters.... a bold strategy Cotton!

@SecurityCRob
Copy link
Contributor

saving money by removing characters.... a bold strategy Cotton!

oh SNAP! unintentional name collision with well-known meme. Double-points to Ravenclaw!

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

Successfully merging this pull request may close these issues.

4 participants