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

[CPLYTM-467] feat: initial work to create assessment results generator #9

Merged
merged 15 commits into from
Jan 31, 2025

Conversation

gvauter
Copy link
Member

@gvauter gvauter commented Jan 23, 2025

Adds feature to generate assessment results from PVP results.

@jpower432 jpower432 self-requested a review January 23, 2025 21:13
jpower432 and others added 5 commits January 23, 2025 16:47
When trying import C2P Go, the operation fails because of the
":" character in the testdata is not valid against go
module.CheckFilePath. Add an empty go.mod will exlude the testdata
when importing the top-level module.

Signed-off-by: Jennifer Power <[email protected]>
fix: add fixes for importing c2p with oscal-sdk-go
@gvauter gvauter marked this pull request as ready for review January 27, 2025 18:15
gvauter and others added 5 commits January 27, 2025 13:35
Signed-off-by: George Vauter <[email protected]>
To allow customization based on implementations for specific
compliance frameworks, a Settings input has been added to GeneratePolicy and
AggregateResults methods to alter the RuleSets passed to plugins based on
settings from a given control implementation.

Signed-off-by: Jennifer Power <[email protected]>
framework/reporter.go Outdated Show resolved Hide resolved
framework/reporter.go Outdated Show resolved Hide resolved
CPLYTM-497: feat: adds Settings to PluginManager operations
Signed-off-by: George Vauter <[email protected]>
Signed-off-by: George Vauter <[email protected]>
@gvauter gvauter requested a review from jpower432 January 30, 2025 17:57
Copy link

@jpower432 jpower432 left a comment

Choose a reason for hiding this comment

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

LGTM

@gvauter gvauter merged commit 31e0f7e into main Jan 31, 2025
1 check passed
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.

2 participants