Skip to content

Conversation

@tim-schilling
Copy link
Member

This removes them as owners from the organization.

Follow-up to #305
Related to django-commons/controls#90

@tim-schilling tim-schilling requested a review from a team as a code owner October 10, 2025 21:06
@cunla
Copy link
Member

cunla commented Oct 12, 2025

I wonder why this did not trigger a plan..
Also, if no-one is an owner - that can cause an issue?

@tim-schilling
Copy link
Member Author

@cunla it's likely because I have the PR based on a non-main branch. I bet it'll trigger once I merge the upstream one.

@tim-schilling tim-schilling force-pushed the reduce-admins-to-moderators branch from 27a529b to 37394ae Compare December 3, 2025 01:38
@tim-schilling tim-schilling force-pushed the reduce-admins-to-moderators branch from 37394ae to f8a4271 Compare December 5, 2025 19:53
@tim-schilling tim-schilling force-pushed the create-ops-team branch 2 times, most recently from 62ffaab to beaa083 Compare December 19, 2025 18:43
Base automatically changed from create-ops-team to main December 19, 2025 18:49
This removes them as owners from the organization.
@tim-schilling tim-schilling force-pushed the reduce-admins-to-moderators branch from f8a4271 to f6c969c Compare December 19, 2025 18:54
@cunla
Copy link
Member

cunla commented Dec 19, 2025

Terraform plan in terraform
With var files: terraform/production/org.tfvars terraform/production/repositories.tfvars
With variables: github_token = (sensitive value)

No changes. Your infrastructure matches the configuration.
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.

📝 Plan generated in Plan org changes and list them in a PR #254

@tim-schilling
Copy link
Member Author

@cunla when you have a chance to review. This should be a no-op

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.

2 participants