Skip to content
This repository has been archived by the owner on Dec 12, 2023. It is now read-only.

Control Implementation Summary (workbook) #138

Merged
merged 3 commits into from
Jul 26, 2021

Conversation

GaryGapinski
Copy link
Collaborator

This PR adds validation rules not previously present, specifically those in §5.3 and §5.3.1.1.

Closes #48.
Closes #101.

The actual "CIS Workbook" will be a separate PR.

@GaryGapinski GaryGapinski self-assigned this Jul 22, 2021
@github-actions

This comment has been minimized.

Copy link

@ohsh6o ohsh6o left a comment

Choose a reason for hiding this comment

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

I think for something like this we should sit together and do a code review. Can you, @danielnaab, and I arrange this tomorrow morning?

@GaryGapinski
Copy link
Collaborator Author

I think for something like this we should sit together and do a code review. Can you, @danielnaab, and I arrange this tomorrow morning?

Perhaps before the ASAP P3 standup or the subsequent NIST OSCAL Model Review

implementation-status.</sch:diagnostic>
<sch:diagnostic doc:assertion="implemented-requirement-has-implementation-status-remarks"
doc:context="oscal:implemented-requirement"
id="implemented-requirement-has-implementation-status-remarks-diagnostic">Thgis incomplete control implementation lacks an
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
id="implemented-requirement-has-implementation-status-remarks-diagnostic">Thgis incomplete control implementation lacks an
id="implemented-requirement-has-implementation-status-remarks-diagnostic">This incomplete control implementation lacks an

@GaryGapinski GaryGapinski merged commit 2e28a01 into develop Jul 26, 2021
@GaryGapinski GaryGapinski deleted the control-implementation-summary branch July 26, 2021 15:47
@github-actions
Copy link

XSpec Test Results

    1 files  ±0    13 suites  ±0   0s ⏱️ ±0s
426 tests ±0  405 ✔️ ±0  21 💤 ±0  0 ❌ ±0 
428 runs  ±0  407 ✔️ ±0  21 💤 ±0  0 ❌ ±0 

Results for commit 2e28a01. ± Comparison against base commit 2e28a01.

danielnaab pushed a commit that referenced this pull request Jul 28, 2021
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.

Check Control Implementation Summary Validate Control Origination
3 participants