promote: test to main (governance doc rename + CI gate fix)#11
Merged
Conversation
* Add CI/CD governance guide and promotion-path enforcement - New fabric-cicd-governance-guide.md: prescriptive guidance for partners on identity, RBAC, branch protection, deployment gating, and audit; out-of-scope concerns demoted to 'things to consider' - New enforce-promotion-path workflow: blocks PRs into test/main that don't originate from dev/test respectively (status check required by the test/main rulesets) - README: link the new governance guide in the Documentation table - workspace_swap.py: clarify ITEM_TYPES comment and remove duplicate type alias * Tighten governance guide and align cross-doc references Governance guide: - Trim duplicated SoD / RBAC / OIDC bullets across rows - Add rollback-strategy row (think-about-it framing) - Clarify path filter (efficiency vs cross-env triggers) - Clarify multi-service UAMI guidance with replicated-resources carve-out - Soften 'immutable' audit claim - Drop hardcoded tenant-setting labels in favor of MS doc link - Add same-security-group caveat for the two SP tenant settings - Add back-link to Implementation Guide README: - Architecture diagram now shows feature -> dev -> test -> main with source-branch restriction notes - Quick Start: drop stale tenant-setting label, add OIDC and Governance Guide cross-links Hybrid Implementation Guide: - Rename H1 to match README link text - Add run-tests.yml and enforce-promotion-path.yml to repo structure - Drop stale tenant-setting label, link to MS docs and Governance Guide * Sharpen README intro and add Git-integration concern + workflow rename - README: lead intro with reference-implementation + solution-accelerator framing, claim end-to-end coverage, and add field-experience byline disclaimer; tighten Who-is-this-for line; drop emphasis bolds - fabric-cicd-release-options.md: add 'Reliance on Fabric Git integration in upper environments' tradeoff to Option 2 (ghost commits, Source Control panel drift) with citation to MS sync/commit limitations - development-swap-to-dev-flow.svg: rename stale 'validate-branch-env.yml' to current 'check-pr-ready.yml' (matches the descriptor box below) * docs: rename governance guide to considerations + soften framing - Rename fabric-cicd-governance-guide.md -> fabric-cicd-governance-considerations.md to better match the doc's consultative posture - Reframe 'Things to consider' section as 'Adjacent controls owned outside the CI/CD pipeline' with required-but-licensing-dependent caveat - README: lead paragraph now calls out developer workflow alongside CI/CD pipeline; Who is this for? adds a pointer for architects/decision-makers evaluating Fabric - Update inbound links in README.md and fabric-hybrid-cicd-guide.md * ci: remove paths filters from required-check workflows Required status checks must report a result for every PR, otherwise the check name stays 'expected' indefinitely and the PR is blocked. The paths: filters on check-pr-ready.yml and run-tests.yml caused exactly this -- doc-only PRs (or Fabric-only PRs) couldn't satisfy the gate because the workflow never ran. Both jobs are short (~30s and ~60-90s) so always-run is acceptable for this repo's velocity. Follows the same pattern as enforce-promotion-path.yml, which never had a paths filter for the same reason.
promote: dev to test (governance doc rename + CI gate fix)
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Promote latest test to main so default branch reflects the new doc layout.
What's in this promotion
fabric-cicd-governance-guide.md->fabric-cicd-governance-considerations.mdand soften framingpaths:filters fromcheck-pr-ready.ymlandrun-tests.ymlDoc-and-CI-only. No Fabric item or deployment changes.