Skip to content

Conversation

@huiwangredhat
Copy link
Collaborator

@huiwangredhat huiwangredhat commented Apr 23, 2025

This is sync oscal->cac. It pairs with the PR for sync cac->oscal .
The test detail:
https://github.com/complytime/oscal-content/actions/runs/14698768504/job/41244469626
The created_pr in cac content is here.

@huiwangredhat huiwangredhat changed the title chore: add sync oscal to cac chore:[CPLYTM-628] add sync oscal to cac Apr 23, 2025
@huiwangredhat huiwangredhat force-pushed the sync_oscal_to_cac branch 6 times, most recently from 049d86a to 4e55791 Compare April 24, 2025 11:38
@huiwangredhat huiwangredhat marked this pull request as ready for review April 24, 2025 11:41
Signed-off-by: Sophia Wang <huiwang@redhat.com>
@huiwangredhat huiwangredhat reopened this Apr 27, 2025
@huiwangredhat huiwangredhat force-pushed the sync_oscal_to_cac branch 2 times, most recently from 307c263 to 7063f70 Compare April 28, 2025 04:25
@huiwangredhat huiwangredhat force-pushed the sync_oscal_to_cac branch 2 times, most recently from 3c20ec2 to 136fc3c Compare April 29, 2025 06:31
Copy link

@hbraswelrh hbraswelrh left a comment

Choose a reason for hiding this comment

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

LGTM. I left a couple comments. The test reference provided looks great.

Signed-off-by: Sophia Wang <huiwang@redhat.com>
@huiwangredhat
Copy link
Collaborator Author

Move it to the draft to hide it in the review list. Since it is no need to review again and it is not meaningful to merge it before moving it to the CAC org.

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