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

chore: trestle release 3.8.0 #1818

Merged
merged 5 commits into from
Feb 13, 2025
Merged

chore: trestle release 3.8.0 #1818

merged 5 commits into from
Feb 13, 2025

Conversation

jpower432
Copy link
Member

Types of changes

  • Hot fix (emergency fix and release)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation (change which affects the documentation site)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Release (develop -> main)

Quality assurance (all should be covered).

  • My code follows the code style of this project.
  • Documentation for my change is up to date?
  • My PR meets testing requirements.
  • All new and existing tests passed.
  • All commits are signed-off.

Summary

Key links:

Before you merge

  • Ensure it is a 'squash commit' if not a release.
  • Ensure CI is currently passing
  • Check sonar. If you are working for a fork a maintainer will reach out, if required.

oscal-compass-bot bot and others added 5 commits January 22, 2025 13:51
* fix: add deprecation warning

Signed-off-by: Lou DeGenaro <[email protected]>

* make lint ahppy

Signed-off-by: Lou DeGenaro <[email protected]>

---------

Signed-off-by: Lou DeGenaro <[email protected]>
* remove submodule nist-source

Signed-off-by: Lou DeGenaro <[email protected]>

* fix: remove nist-source references

Signed-off-by: Lou DeGenaro <[email protected]>

---------

Signed-off-by: Lou DeGenaro <[email protected]>
* feat: upgrade to OSCAL 1.1.3

Signed-off-by: Lou DeGenaro <[email protected]>

* fix: update README to OSCAL 1.1.3

Signed-off-by: Lou DeGenaro <[email protected]>

---------

Signed-off-by: Lou DeGenaro <[email protected]>
* fix issue 1797

Signed-off-by: Lou DeGenaro <[email protected]>

* fix: merge header for prof when ohv specified

Signed-off-by: Lou DeGenaro <[email protected]>

* fix: remove profile header merge restriction

Signed-off-by: Lou DeGenaro <[email protected]>

* fix: test_profile_ohv

* fix bdist

Signed-off-by: Lou DeGenaro <[email protected]>

* remove debug code

Signed-off-by: Lou DeGenaro <[email protected]>

* fix: assure sorted by param_id

Signed-off-by: Lou DeGenaro <[email protected]>

* disable prof_test ohv else clause (for now)

Signed-off-by: Lou DeGenaro <[email protected]>

* use simple_test_profile2.json for proftest ohv

Signed-off-by: Lou DeGenaro <[email protected]>

* prof_test sort by pram_id for test ohv

Signed-off-by: Lou DeGenaro <[email protected]>

* add sort to test_profile_values_included_if_replaced

Signed-off-by: Lou DeGenaro <[email protected]>

---------

Signed-off-by: Lou DeGenaro <[email protected]>
Copy link
Member

@vikas-agarwal76 vikas-agarwal76 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@degenaro degenaro left a comment

Choose a reason for hiding this comment

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

LGTM

@jpower432 jpower432 marked this pull request as ready for review February 13, 2025 21:00
@jpower432 jpower432 requested a review from a team as a code owner February 13, 2025 21:00
@jpower432 jpower432 merged commit 815d74b into main Feb 13, 2025
29 checks 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.

3 participants