Skip to content

chore(governance): stop declaring repository identity in settings.yml - #222

Merged
hyperpolymath merged 1 commit into
mainfrom
chore/settings-no-identity
Sep 19, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
chore/settings-no-identity

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

probot/settings applies this file on every push to the default branch, so the four identity
keys it declared are instructions, not documentation. The estate has already paid for exactly this:
.github/settings.yml once read name: "{{REPO}}", GitHub collapsed the illegal braces to dashes,
the repository renamed itself to -REPO- on every push, its old URL 404'd, and it was read as a
deleted repository.

This repo still declared name, description, homepage and private.

The repair is convergence on the source of truth, not new content:

  • the header becomes the template's own header, which documents the incident and the rule
    ("THIS FILE MUST NEVER DECLARE REPOSITORY IDENTITY");
  • the four identity keys are removed; everything else is unchanged — same repository settings,
    same 18 labels;
  • repository identity and visibility stay set out of band, once, by the owner.

Verified with the estate's own gate, scripts/check-no-placeholders.sh, whose settings.yml guard
runs in template repos too (that exemption is how the original incident went unseen). On this
branch the guard reports clean.

probot/settings applies this file on every push to the default branch, so `name`,
`description`, `homepage` and `private` here are instructions, not documentation. The estate has
already paid for that: the template carried `name: "{{REPO}}"`, GitHub collapsed the illegal braces
to dashes, the repo renamed itself to `-REPO-` on every push and its old URL 404'd.

This file now matches the template's own header and rule, with the four identity keys removed and
everything else left as it was. Repository identity and visibility are set out of band, once, by
the owner. Enforced by scripts/check-no-placeholders.sh.
@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 7d76d40c-5977-40cf-80a5-b9c24db8454d

📥 Commits

Reviewing files that changed from the base of the PR and between 867bd6a and 8f6c739.

📒 Files selected for processing (1)
  • .github/settings.yml
 _________________________________________________________
< Eliminate bugs. Ship early. Sleep well. Pick all three. >
 ---------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath merged commit bdee8cc into main Sep 19, 2026
26 of 32 checks passed
@hyperpolymath
hyperpolymath deleted the chore/settings-no-identity branch September 19, 2026 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant