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.

Sign up for GitHub

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

Jump to bottom

Oscal data analysis #13

Merged
merged 10 commits into from
Mar 24, 2024
Merged

Oscal data analysis #13

merged 10 commits into from
Mar 24, 2024

Conversation

sarnold
Copy link
Owner

@sarnold sarnold commented Mar 18, 2024

  • add new module and support code

@sarnold
chg: cleanup readme and tox file bits !minor
6bc7e47 
Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@sarnold
new: dev: add stub module for oscal data, update packaging and tox files
ce04457 
* add .repolite.yml for syncing oscal-content repo
* update tests and vendor tox plugin for shared envs

Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@pep8speaks
Copy link

pep8speaks commented Mar 18, 2024 •
edited
Loading

Hello @sarnold! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 67:91: E501 line too long (91 > 90 characters)
Line 30:91: E501 line too long (99 > 90 characters)

Comment last updated at 2024-03-23 23:01:17 UTC

@sarnold
chg: dev: flesh out argparse and yaml config keys
e2bfc75 
Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@sarnold
new: dev: test fuzzy-match for analyze script, consolidate code
d11e3dc 
* limit fuzzy match results using startswith input ID string
* cleanup package modules and tests, move initial profile funcs

Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@sarnold
add small set of test IDs from openembedded profile
22691af 
Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@sarnold
new: add file reader to handle csv, json, yaml, and simple text files
bf5730d 
* for each type, parse into a list of objects and return the list
* where simple text files are something like the control IDs
  in tests/data

Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@sarnold
chg: add another text_file_reader consumer and cleanup some lint
161be8d 
Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@sarnold
chg: bump all workflow action versions
c48f324 
Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@sarnold
fix: doc: cleanup docstrings after sphinx checks
1c06880 
Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@sarnold
chg: bump tox workflow deps and cleanup/improve readme
0b21adc 
Signed-off-by: Steve Arnold <sarnold@vctlabs.com>
@github-actions GitHub Actions
Copy link

Package Line Rate Branch Rate Complexity Health
src.ymltoxml 96% 91% 0 ✔
Summary 96% (237 / 247) 91% (78 / 86) 0 ✔

@github-actions GitHub Actions
Copy link

Hello @sarnold! Thanks for opening this PR. We found the following information based on analysis of the coverage report:

Base Branch Rate coverage is 85%
Merging d62e1c9 into main will increase coverage by 6%

Nice work, @sarnold. Cheers! 🍻

@sarnold sarnold merged commit adc965b into main Mar 24, 2024
34 checks passed
@sarnold sarnold deleted the oscal-data branch March 24, 2024 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sarnold @pep8speaks