Skip to content

Let the Rumble moderator team review escalations and welcome volunteers - #30

Merged
flemming-n-larsen merged 3 commits into
mainfrom
moderator-team-governance
Oct 22, 2026
Merged

flemming-n-larsen merged 3 commits into
mainfrom
moderator-team-governance

Conversation

@flemming-n-larsen

@flemming-n-larsen flemming-n-larsen commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Purpose

Give human Rumble moderation to the existing robocode-dev/rumble-moderators team so qualified volunteers can share reviews, disputes, and recovery work.

Changes

  • Route escalations and protected-file ownership to the team.
  • Check active team membership before /moderation approve <reason> can clear a pending status for the current PR head. A failed lookup cannot approve, and a human-cleared status never merges automatically.
  • Add a public volunteer issue form and document review skills, conflicts of interest, and onboarding.
  • Link Git for Windows and the permitted bot licenses in the contributor guides.

Verification

  • python -m unittest discover -s tests -p test_moderation.py — 19 passed.
  • git diff --check — clean.
  • On 2026-10-22, GitHub's organization-installations API reported members: read for Moderator App installation 173538338, and the team-repository listing reported Write access to rumble-bots.
  • The team's current maintainer returned active from GitHub's membership endpoint under the maintainer's token. The installation-token lookup remains unexercised until the trusted workflow contains this PR's code.

Before merging

  • Keep the required bot-moderation status, review and branch rules in place.
  • Merge by a human reviewer; this PR changes moderation policy and workflow code. After merge, exercise the team-authorized command on a pending escalation using the App installation token.

@tank-royale-rumble-moderator

Copy link
Copy Markdown
Contributor

@flemming-n-larsen Moderation needs your decision for 009eb27437a6: Draft PR or missing verdict: Non-bot or protected file changed: .github/ISSUE_TEMPLATE/moderator-volunteer.yml

The required check stays pending. After reviewing, use /moderation approve <reason> or request changes.

@tank-royale-rumble-moderator

Copy link
Copy Markdown
Contributor

@flemming-n-larsen Moderation needs your decision for 170af612c8c2: Draft PR or missing verdict: Non-bot or protected file changed: .github/ISSUE_TEMPLATE/moderator-volunteer.yml

The required check stays pending. After reviewing, use /moderation approve <reason> or request changes.

@flemming-n-larsen
flemming-n-larsen marked this pull request as ready for review October 22, 2026 18:01
@tank-royale-rumble-moderator

Copy link
Copy Markdown
Contributor

@flemming-n-larsen Moderation needs your decision for 170af612c8c2: Non-bot or protected file changed: .github/ISSUE_TEMPLATE/moderator-volunteer.yml

The required check stays pending. After reviewing, use /moderation approve <reason> or request changes.

@tank-royale-rumble-moderator

Copy link
Copy Markdown
Contributor

@flemming-n-larsen Moderation needs your decision for 8a196b8fdbcb: Non-bot or protected file changed: .github/ISSUE_TEMPLATE/moderator-volunteer.yml

The required check stays pending. After reviewing, use /moderation approve <reason> or request changes.

@flemming-n-larsen
flemming-n-larsen merged commit 01e3d8d into main Oct 22, 2026
3 of 4 checks passed
@flemming-n-larsen
flemming-n-larsen deleted the moderator-team-governance branch October 22, 2026 18:23
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