Repository navigation
docs(sonar): resolve stale org comment contradicting its own values - #62
Merged
Merged
Conversation
PR #35 recorded the SonarCloud organisation as an unverified assumption favouring metadatastician. PR #42 decided it as hyperpolymath and changed the values but not the comment, so the file has contradicted itself since 2026-07-23: - the header Project URL pointed at metadatastician_IDApTIK while sonar.projectKey declares hyperpolymath_IDApTIK - the comment still read "ORG IS ASSUMED, NOT VERIFIED" and framed hyperpolymath / hyperpolymath_IDApTIK as a hypothetical alternative to the exact values printed two lines below it State the decision as settled, citing #42 and #38, and keep the two durable estate warnings (stale template key; fix projectKey before adding SONAR_TOKEN, since a wrong key plus a valid token reports silently into another project). Records only durable facts, not transient status -- recording status in a committed comment is what let the original rot. Values unchanged. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
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.
Summary
sonar-project.propertiescontradicted itself. Comment-only fix —sonar.organizationandsonar.projectKeyare unchanged.PR #35 added a comment block recording the SonarCloud organisation as an unverified assumption, with
metadatasticianas the guess. PR #42 then decided the organisation ashyperpolymathand changed the values — but not the comment. The two have disagreed onmainsince 2026-07-23.Two concrete defects:
id=metadatastician_IDApTIKwhile line 16 declaressonar.projectKey=hyperpolymath_IDApTIK. The header URL led to the wrong project.ORG IS ASSUMED, NOT VERIFIED -- maintainer to confirm before provisioning, and argued that "if the estate in fact consolidates under the hyperpolymath SonarCloud org, this should read hyperpolymath / hyperpolymath_IDApTIK instead" — describing as a hypothetical the exact two values printed immediately below it. Anyone reading the file was told the question was open when it had been settled four days earlier.Both were mine, from #33/#35. Sorry for the noise.
What the comment says now
hyperpolymath, decided in Worktree idaptik governance alignment #42, closing Decide the SonarCloud organisation for metadatastician-org repos #38 — stated as fact, with the reasoning that the GitHub-org boundary deliberately does not mirror the SonarCloud one (the thing I originally got wrong).hyperpolymath_rsr-template-repokey in template-derived repos, and the ordering rule — fix a wrongprojectKeybefore addingSONAR_TOKEN, because without the token a scan fails at authentication and submits nothing, whereas with the token and a wrong key it silently reports into another project.A deliberate choice about wording
The original comment rotted because it recorded transient status ("assumed, not verified", "confirm before provisioning") in a committed file, where it then outlived the decision. So this version deliberately records only durable facts.
Notably, it does not say the project is unprovisioned or the token unset — both true right now (verified:
hyperpolymath_IDApTIKreturns not-found, noSONAR_TOKENon this repo), and both things that stop being true the moment you set them up. Those belong in #38 and the dev-note, not here. The file states the requirement instead, which stays accurate either way.Test plan
git diffconfirms nosonar.*value line changed — comments onlysonar.projectKeymain(9e86676)Unrelated and still outstanding:
metadatastician/cadastradeclareshyperpolymath_rsr-template-repowhile its README badge saysmetadatastician_cadastra. Now that the org is settled, its config org is right, its key is wrong, and the badge should becomehyperpolymath_cadastra. Happy to do that separately.🤖 Generated with Claude Code