Skip to content

chore: remove dead code and document repository management - #163

Merged
thomasgarrison merged 1 commit into
mainfrom
chore/cleanup-dead-code
Jun 11, 2026
Merged

chore: remove dead code and document repository management#163
thomasgarrison merged 1 commit into
mainfrom
chore/cleanup-dead-code

Conversation

@thomasgarrison

Copy link
Copy Markdown
Contributor

Summary

Cleanup-only PR, no resource changes expected (plan should be empty apart from refresh):

  • Delete the empty src/moved.tf left over from a long-completed state migration.
  • Remove the try(local.webhooks["embedded"], {}) lookups in repositories_embedded.tf and repositories_templates.tf — no embedded webhook is defined, so the expression always silently resolved to {}. Now it says {} outright.
  • README: replace the outdated repository type list (lib-*-rust/svc-*-rust no longer exist) with the actual repository groups, document the managed vs seeded file mechanisms introduced in fix: stop provisioning cspell config #162, and fill in the long-standing 'Adding a new repository' placeholder — including the cross-repo ordering constraint that teams must exist in tf-onboarding before a repository here can reference them.

🤖 Generated with Claude Code

- Delete the empty moved.tf left over from a completed state migration
- Drop the webhooks["embedded"] lookups; no such webhook is defined,
  the try() always resolved to an empty map
- README: replace the outdated repository type list, document the
  managed vs seeded file mechanisms, and fill in the 'Adding a new
  repository' section (including the tf-onboarding team ordering)
@thomasgarrison
thomasgarrison requested a review from a team as a code owner June 11, 2026 21:59
@cla-bot cla-bot Bot added the cla-signed label Jun 11, 2026
@github-actions

Copy link
Copy Markdown

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖success

Validation Output
Success! The configuration is valid.


Terraform Plan 📖success

Show Plan
terraform

No changes. Your infrastructure matches the configuration.

Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.

Pusher: @thomasgarrison, Action: pull_request, Working Directory: ``, Workflow: Terraform

@thomasgarrison
thomasgarrison merged commit 59f55d5 into main Jun 11, 2026
5 checks passed
@thomasgarrison
thomasgarrison deleted the chore/cleanup-dead-code branch June 11, 2026 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant