This repository has been archived by the owner on Dec 12, 2023. It is now read-only.
forked from GSA/fedramp-automation
-
Notifications
You must be signed in to change notification settings - Fork 6
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
ohsh6o
force-pushed
the
issue-39-oscal-rc1-fixups
branch
from
January 12, 2021 17:02
ea0b261
to
f6e9842
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Merged
Since 1.0.0-rc1 introduced some breaking changes, we need to check for this now. The new upgraded example SSP does not have that field edited since I believe it was manually created and/or manually edited after it was run through the usnistgov/OSCAL M3->RC1 upconverter, this seems to have been missed.
Put them in the right locations in the report instead of repeating them ad nauseam.
This is annoying and distoring module updates.
We will need to make way for reading values from upstream OSCAL Metaschema files.
It can only fire once, which is why tests were breaking.
ohsh6o
force-pushed
the
issue-39-oscal-rc1-fixups
branch
from
January 18, 2021 14:53
180e2da
to
42c5cec
Compare
During the rebase and what looked like the last commit, this somehow got lost not once, but twice!
Will add tests then use it for context filter.
Passing in the @ns attribute as just a string won't work.
ohsh6o
changed the title
More Fixes for 1.0.0-rc1 Release
More Fixes for 1.0.0-rc2 Release
Apr 8, 2021
XSpec Test Results 1 files ±0 2 suites ±0 0s ⏱️ ±0s Results for commit 0c3de86. ± Comparison against base commit 28978ae. This pull request removes 2 and adds 11 tests. Note that renamed tests count towards both.
|
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This time, these changes will collect the last remaining bits and pieces for the move to 1.0.0-rc1.
Closes #39.