Skip to content

Combine member updates and guarded deletes - #265

Draft
galargh wants to merge 15 commits into
masterfrom
galargh/combine-pr-263-264
Draft

Combine member updates and guarded deletes#265
galargh wants to merge 15 commits into
masterfrom
galargh/combine-pr-263-264

Conversation

@galargh

@galargh galargh commented Aug 9, 2026

Copy link
Copy Markdown
Member

Summary

This draft PR combines the work from:

It also includes follow-up improvements introduced in this branch:

  • Keep routine Fix PR comments focused on access changes only, while publishing the full access report as the Fix workflow summary/artifact.
  • Link access-change comments to the Fix workflow run so reviewers can inspect the full access breakdown when needed.
  • Explain repository access paths in access changes and reports, distinguishing direct access from team-granted access.
  • Flag members with no effective repository access as potential no members, even if they are in teams without repository access.
  • Fail Plan/Apply classification when removing an organization member would leave team membership or direct private repository access behind.
  • Require ALLOW_DESTROY=true on allow-destroy environments before disabling Terraform prevent_destroy.
  • Document the access changes action update requirement and allow-destroy environment variable setup.

Validation

  • pnpm --dir scripts install --frozen-lockfile
  • pnpm --dir scripts run format
  • pnpm --dir scripts run lint
  • pnpm --dir scripts run build
  • pnpm --dir scripts test

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