Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

Feat/oscal cli action #584

Closed
wants to merge 7 commits into from
Closed

Feat/oscal cli action #584

wants to merge 7 commits into from

Conversation

CloudBeard
Copy link
Contributor

Add OSCAL CLI Action to validate OSCAL Schema.

Updated OSCAL config files to include additional party in metadata section (P1 upstream)

@CloudBeard CloudBeard requested review from mjnagel and MxNxPx and removed request for mjnagel October 2, 2023 18:59
runs-on: ubuntu-latest

steps:
- name: Stepup Java
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- name: Stepup Java
- name: Setup Java

typo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My spell checker didn't catch it. :(

mjnagel
mjnagel previously approved these changes Oct 2, 2023
Copy link
Collaborator

@mjnagel mjnagel left a comment

Choose a reason for hiding this comment

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

LGTM otherwise

@CloudBeard
Copy link
Contributor Author

OSCAL-CLI has a bug. Will reopen after next OSCAL-CLI version and additional testing.

@CloudBeard CloudBeard closed this Oct 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants