Repository navigation
chore: configure Core central Sonar policy and merged runner pin - #26
Merged
Merged
Conversation
This was referenced Oct 22, 2026
This was referenced Oct 25, 2026
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
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.
Problem and result
Core's merged policy has an empty profile inventory, which correctly rejects central analysis before upload. This PR records the actual 27 dedicated
Core Library 1.0.0profile snapshots provisioned on the central service, including 121 C# rules and each profile's rule revision timestamp. The runbook records the completed project/access setup and the remaining activation steps.The Sonar caller also needs a reachable merged runner reference. The first main Sonar trigger failed workflow validation before starting any job: GitHub reported that the reusable workflow at
64b791bc90feadfd21a8cc3cf73b14456172334bwas not found. Both source commits have the identical tree9d50d51b4e2fc7d0fefba61d746ab93720a54bef, and the workflow file has the same blob. This PR moves all three coupled references (caller, tool input and policy) to merged main revisionf6c806d1060e390a24c1451e7486578c9bdbab23. This narrowly scoped Action change was explicitly approved. Hosted resolution still needs verification after merge; static validation alone does not prove it.Three files change: the policy, runbook and the two SHA values in
sonar-main.yml. All other Actions, the six gate conditions, null baseline and library/tests/dependencies remain unchanged.Provisioned and verified
coreproject, dedicated gate and independent copied profiles with no parent. Accepted CSharpier-compatible rules retained; default profiles and app-foundation's profiles/baseline unchanged.sonar-mainenvironment:mainbranch only, André as required reviewer, self-approval allowed, administrator bypass disabled. Only the two Core Sonar secrets are stored there.Validation
e553968869d1e248b61fb1090cbfd69178460a88; publishing skipped.f6c806d1060e390a24c1451e7486578c9bdbab23.3703d9d7b7e9dc08e97c2938634c74d711c664f3; GitHub reports the build-and-test check completed successfully and the PR is open for review.Remaining rollout
SONAR_MAIN_ENABLEDstays absent andbaselineAnalysisIdstays null. Review and merge this actual profile snapshot before controlled activation. The first exact-main analysis must then be reviewed as a baseline; its missing-baseline readiness failure is expected. Record the selected baseline through review, then collect two fresh exact-main CI/Sonar successes and capacity evidence under analysis load.Refs #24 and andregoepel/app-foundation#238. Keep the rollout issues open. No merge, release tag, upload, publishing activation, deployment, other-repository source change or further Action change is part of this PR. Core's existing publisher remains outside these Sonar controls.