-
Notifications
You must be signed in to change notification settings - Fork 1
Add governance and change management primitives with YAML/JSON configuration support #16
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
Conversation
…ersioning, and policy features Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com>
Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com>
…ate registrations Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com>
|
@copilot We need comprehensive documentation added to docs under the subheading of governance to explain how, where, and why you'd use this library. We need to be complete with the samples and details. |
JerrettDavis
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need comprehensive documentation added to our docs for everything that has been added.
Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com>
Added comprehensive governance documentation in
Also added a working sample in
The documentation is integrated into the existing docs structure under "Administration & Governance" section. Commit: 08ed627 |
…ocumentation Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com>
Implemented comprehensive DSL support for governance configuration via YAML/JSON: Configuration Support Added:
Documentation:
Tests:
Example YAML: experimentFramework:
governance:
approvalGates:
- type: roleBased
fromState: Approved
toState: Running
allowedRoles: [operator, sre]
policies:
- type: trafficLimit
maxTrafficPercentage: 10.0
minStableTime: '00:30:00'Commit: cf97ae1 |
tests/ExperimentFramework.Tests/Configuration/Governance/GovernanceConfigurationTests.cs
Outdated
Show resolved
Hide resolved
tests/ExperimentFramework.Tests/Configuration/Governance/GovernanceConfigurationTests.cs
Outdated
Show resolved
Hide resolved
…infrastructure Co-authored-by: JerrettDavis <2610199+JerrettDavis@users.noreply.github.com>
Code Coverage |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #16 +/- ##
=======================================
Coverage ? 84.94%
=======================================
Files ? 170
Lines ? 6178
Branches ? 883
=======================================
Hits ? 5248
Misses ? 930
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Governance Tests - All Passing ✅
Fixed Skipped Tests
Skipattribute from integration testsExperimentConfigurationLoaderIDisposablepattern for temp directory cleanupTest Coverage
Implementation Details
ExperimentConfigurationLoader.LoadFromFile()to properly load YAML/JSONTest Results
All governance configuration tests now run and validate the complete YAML/JSON DSL integration.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.