feat(ycst-org-uk): move the two ycst repos out of yo61 - #66
Merged
Conversation
There was a problem hiding this comment.
Renames+relocates the two ycst-* repos' data files under data/ycst-org-uk/ and adds the two moved blocks that carry each repo's four state instances across the provider alias and renamed for_each key. Collaborators switch from a direct PlanetSeth grant to the admins team, which already exists in data/ycst-org-uk/_teams.yaml with planetseth (lowercase) as a member. filename/name: fields, for_each keys, and moved block addresses all line up correctly, and the PR body's own pre-merge sequencing (transfer/rename on GitHub before merging, unscoped plan) is the right call given moved blocks can't be targeted. No issues found.
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.
Moves
ycst-admin-docsandycst-website-testingfromyo61toycst-org-ukas
board-docsandwebsite-testing, and grants admin through theadminsteam on both instead of naming
PlanetSethdirectly.Do not merge yet. The repos must be transferred and renamed on GitHub
first.
github_repositorytakes its owner from the provider, so Terraformcannot do the move; one
movedblock per repo then carries all four of itsstate instances across the provider alias and the renamed
for_eachkey.Merging before the transfer creates a window where the plan proposes to
create
ycst-org-uk/board-docs; applying that would leave an empty repo whosename then blocks the real transfer. Transferring first inverts the failure into
a harmless plan against
yo61, since the API redirects the old paths.The plan for this must be unscoped — a cross-org
movedblock cannot betargeted to one side (
Moved resource instances excluded by targeting).Expected plan, and nothing else:
github_repository_collaborators.this:board-docsdropsPlanetSeth's direct grant for the team grant, andwebsite-testinggainsits first team block, having had no collaborators block at all
auto_init: trueis kept onwebsite-testingeven though it only has meaningat creation: it is in state, and dropping it would show a diff for no
behavioural change.
visibility: privatelikewise restates the module defaultand is kept, since it is the fact the access-control design rests on.
Design:
docs/superpowers/specs/2026-08-12-ycst-org-uk-migration-design.mdPlan:
docs/superpowers/plans/2026-08-13-ycst-org-uk-migration.md(Tasks 5-7)🤖 Generated with Claude Code
https://claude.ai/code/session_01C15usdmazQC71xZC1YFGHg