Skip to content

Remove repo_osac_templates module configuration#61

Open
adriengentil wants to merge 1 commit intomainfrom
adriengentil-patch-1
Open

Remove repo_osac_templates module configuration#61
adriengentil wants to merge 1 commit intomainfrom
adriengentil-patch-1

Conversation

@adriengentil
Copy link
Copy Markdown
Contributor

@adriengentil adriengentil commented Mar 27, 2026

Removed the 'repo_osac_templates' module configuration from repositories.tf.

osac-templates is now archived, and its content was moved to osac-aap. It was moved because it removes re-vendoring friction.

Summary by CodeRabbit

  • Chores
    • Removed osac-templates repository configuration from managed infrastructure.

Removed the 'repo_osac_templates' module configuration from repositories.tf.

`osac-templates` is now archived, and its content was moved to `osac-aap`. It was moved because it removes re-vendoring friction.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 91c20b22-b1ec-4786-a547-4bb9b1673c66

📥 Commits

Reviewing files that changed from the base of the PR and between 8549723 and 2a53fe6.

📒 Files selected for processing (1)
  • repositories.tf
💤 Files with no reviewable changes (1)
  • repositories.tf

Walkthrough

A Terraform module block for the "osac-templates" repository was removed from repositories.tf. The module defined a public repository with fulfillment-wg team permissions set to admin access and specific repository policies including a required status check for "ci/prow/temp". All other repository module definitions remain unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately describes the main change: removing the repo_osac_templates module configuration from repositories.tf.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch adriengentil-patch-1

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 and usage tips.

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