Skip to content

Remove Alex-Welsh from GitHub teams - #588

Merged
cityofships merged 1 commit into
mainfrom
rm-alex
Nov 6, 2026
Merged

cityofships merged 1 commit into
mainfrom
rm-alex

Conversation

@Alex-Welsh

Copy link
Copy Markdown
Member

Or I'll hack all your mainframes

@Alex-Welsh
Alex-Welsh requested a review from a team as a code owner November 6, 2026 16:14
@coderabbitai

coderabbitai Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Explain Change →

Summary

Removed Alex-Welsh from the Admins, Ansible, Developers, Kayobe and ReleaseTrain team member lists in terraform/github/terraform.tfvars.json.

The Terraform GitHub configuration removes Alex-Welsh from the Admins, Ansible, Developers, Kayobe and ReleaseTrain teams.

Changes: Team membership update

Layer / File(s) Summary
Remove team memberships
terraform/github/terraform.tfvars.json
Removed Alex-Welsh from the Admins, Ansible, Developers, Kayobe and ReleaseTrain team member lists.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: Moderate · up to b9522

These changes alone do not establish complete offboarding. Confirm organisation removal and verify that no team memberships remain before merging.

Pre-merge checks: Passed 3

Check name Status Explanation
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files.

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

Review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 50f93c3 and b9522e4.

Files selected for processing: 1

  • terraform/github/terraform.tfvars.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Comment thread terraform/github/terraform.tfvars.json Resolved
@Alex-Welsh Alex-Welsh changed the title Remove Alex-Welsh from GitHub org Remove Alex-Welsh from GitHub teams Nov 6, 2026
@stackhpc-ci

Copy link
Copy Markdown
Contributor

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖success

Validation Output

Success! The configuration is valid.


Terraform Plan 📖success

Show Plan

undefined

Pusher: @Alex-Welsh, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

@cityofships
cityofships merged commit bcc7c8c into main Nov 6, 2026
6 checks passed
@cityofships
cityofships deleted the rm-alex branch November 6, 2026 16:50
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.

5 participants