Skip to content
This repository was archived by the owner on Jul 1, 2025. It is now read-only.

Feature/canary constraints #1172

Merged
merged 6 commits into from
Feb 25, 2025

Conversation

wandmagic
Copy link

Committer Notes

add constraints that were missing from canary

All Submissions:

By submitting a pull request, you are agreeing to provide this contribution under the CC0 1.0 Universal public domain dedication.

@wandmagic wandmagic requested a review from a team as a code owner February 21, 2025 17:25
@aj-stein-gsa
Copy link
Contributor

Thanks for the rebase, @wandmagic, I will look in a bit.

Gabeblis
Gabeblis previously approved these changes Feb 24, 2025
@aj-stein-gsa aj-stein-gsa mentioned this pull request Feb 24, 2025
7 tasks
Copy link
Contributor

@aj-stein-gsa aj-stein-gsa left a comment

Choose a reason for hiding this comment

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

Is there a reason on /system-security-plan/system-implementation[1]/inventory-item[1] we are not adding back in the necessary prop[@name="software-name"] and for /system-security-plan/system-implementation[1]/inventory-item[4] the inventory item prop[@name="function"], @wandmagic? I might just push up and bull this in to get it over with.

@wandmagic
Copy link
Author

Is there a reason on /system-security-plan/system-implementation[1]/inventory-item[1] we are not adding back in the necessary prop[@name="software-name"] and for /system-security-plan/system-implementation[1]/inventory-item[4] the inventory item prop[@name="function"], @wandmagic? I might just push up and bull this in to get it over with.

the canary and develop got pretty out of sync, but i think we now have the missing constraints passing, so we should be good, thanks for your help!

@aj-stein-gsa
Copy link
Contributor

the canary and develop got pretty out of sync, but i think we now have the missing constraints passing, so we should be good, thanks for your help!

Ha, well I was like "no they aren't" and I see you picked up where I left off before sleepy time. Nice work!

@aj-stein-gsa aj-stein-gsa merged commit f2be082 into GSA:develop Feb 25, 2025
6 checks passed
@wandmagic
Copy link
Author

the canary and develop got pretty out of sync, but i think we now have the missing constraints passing, so we should be good, thanks for your help!

Ha, well I was like "no they aren't" and I see you picked up where I left off before sleepy time. Nice work!

Teamwork makes the dreamwork

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.

3 participants