Skip to content

[WIP] Update direct-push guidance for non-bypass users - #192

Merged
BigSimmo merged 3 commits into
mainfrom
claude/scope-direct-push-guidance
Jul 2, 2026
Merged

[WIP] Update direct-push guidance for non-bypass users#192
BigSimmo merged 3 commits into
mainfrom
claude/scope-direct-push-guidance

Conversation

@Claude

@Claude Claude AI commented Jul 2, 2026

Copy link
Copy Markdown

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

This section details on the original issue you should resolve

<issue_title>P2 Badge Scope direct-push guidance to non-bypass users</issue_title>
<issue_description>P2 Badge Scope direct-push guidance to non-bypass users

In the documented configuration where enforce_admins is OFF and the admin bypass is intentionally retained, the stated consequence that direct pushes to main are blocked for all work is not accurate for repository admins using that bypass. Since this file is the process source of truth for preventing red merges, this should either say direct pushes are blocked for non-bypass actors or the protection should actually enforce admins if the intended rule is no direct pushes by anyone.

Useful? React with 👍 / 👎.

Originally posted by @chatgpt-codex-connector[bot] in #189 (comment)</issue_description>

<agent_instructions>Fix this issue please and commit </agent_instructions>

Comments on the Issue (you are @claude[agent] in this section)

@Claude Claude AI assigned Claude and BigSimmo Jul 2, 2026
@Claude Claude AI linked an issue Jul 2, 2026 that may be closed by this pull request
**<sub><sub>![P2 Badge](https://img.shields.io/badge/P2-yellow?style=flat)</sub></sub> Scope direct-push guidance to non-bypass users** #191
Closed
@Claude @BigSimmo
docs: scope direct-push guidance to non-bypass users
630577d 
Clarify that branch protection blocks direct pushes for non-bypass
users only; admins retaining the bypass (enforce_admins OFF) can still
push directly but should prefer PRs to preserve the required-check gate.

Closes #191

Agent-Logs-Url: https://github.com/BigSimmo/Database/sessions/50ee625e-b9ae-4c5d-8c25-e6182e321abe

Co-authored-by: BigSimmo <87357024+BigSimmo@users.noreply.github.com>
@Claude
Claude AI requested a review from BigSimmo July 2, 2026 14:42
Copilot stopped work on behalf of BigSimmo due to an error July 2, 2026 14:42
Claude Code process exited with code 1
@BigSimmo
BigSimmo marked this pull request as ready for review July 2, 2026 14:44
@BigSimmo
BigSimmo enabled auto-merge (squash) July 2, 2026 14:45
BigSimmo
BigSimmo reviewed Jul 2, 2026
View reviewed changes

@BigSimmo BigSimmo left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Nil

@BigSimmo
BigSimmo merged commit 14fd3ba into main Jul 2, 2026
4 checks passed
@BigSimmo
BigSimmo deleted the claude/scope-direct-push-guidance branch July 2, 2026 16:31
@BigSimmo BigSimmo mentioned this pull request Aug 14, 2026
Merged
3 tasks
BigSimmo pushed a commit that referenced this pull request Aug 14, 2026
@claude
docs(ledger): record PR #1964 review (ledger #192 / maturity X6)
7821558 
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QLbw9qpfjv5CeNz6XpmteN
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BigSimmo BigSimmo BigSimmo left review comments

Assignees

@BigSimmo BigSimmo

@Claude Claude

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

**<sub><sub>![P2 Badge](https://img.shields.io/badge/P2-yellow?style=flat)</sub></sub> Scope direct-push guidance to non-bypass users**

2 participants

@Claude @BigSimmo