Skip to content

Fix docs claiming main is protected - #3

Merged
Tivo0921 merged 1 commit into
mainfrom
docs/fix-branch-protection-claims
Aug 22, 2026
Merged

Tivo0921 merged 1 commit into
mainfrom
docs/fix-branch-protection-claims

Conversation

@Tivo0921

Copy link
Copy Markdown
Contributor

What

教材が「main はBranch Protectionで保護されている / 直接Pushは拒否される」と断言していた箇所を、実態に合わせて修正しました。

Why

Branch Protection を設定しようとしたところ、403 で拒否されました。

{"message": "Upgrade to GitHub Pro or make this repository public to enable this feature.", "status": "403"}

Organization Lumos-Programming が GitHub Free プラン、このRepositoryが Private のためです。Rulesets も同様に利用できません。

特に危険だったのが実習1の Step 11.5 です。「main へ直接Pushを試して、拒否されることを確認する」という手順でしたが、実際には保護されていないため Pushが成功して main に空Commitが入り、研修環境が壊れます。

Changes

  • README.md: 保護されていない事実と、ルールとして各自が守る必要があることを明記
  • docs/02-rules.md: Rule 2 を「実務ではこう守られる」と「ここでは保護されていない」に分離
  • exercises/01-clone-to-pr.md: Step 11.5 を実演からスキップ指示に変更
  • .github/ISSUE_TEMPLATE/training.yml: 対応するチェック項目を修正
  • docs/admin-setup.md: 制約の詳細と、有効化するための2つの選択肢(Public化 / Teamプラン)を記録

How to Test

  • CI(Training Check)が緑になること
  • README・02-rules・実習1 に「保護されている」という断定が残っていないこと

Check

  • 自分で動作確認した
  • git diff を見て、不要な変更が入っていないことを確認した
  • SecretをCommitしていない
  • Commit Messageが「何を変更したか」分かる内容になっている
  • 1つのPull Requestに複数の目的を混ぜていない

Related

Refs #2

Branch Protection を有効にする場合は、この変更を一部元に戻す必要があります。手順と判断材料は docs/admin-setup.md に記載しました。

Organization が GitHub Free プラン かつ Private Repository のため、
Branch Protection / Rulesets を有効化できないことが判明した(API は 403)。

しかし README・02-rules・実習1 は「mainは保護されている」「直接Pushは拒否される」
と断言しており、実態と食い違っていた。特に実習1 Step 11.5 は直接Pushを
試させる手順だったため、そのままでは main に空Commitが入り研修環境が壊れる。

- README: 保護されていない旨と、ルールとして守る必要があることを明記
- docs/02-rules.md: Rule 2 を「実務ではこうなる」+「ここでは保護されていない」に分離
- exercises/01-clone-to-pr.md: Step 11.5 を実演からスキップ指示に変更
- .github/ISSUE_TEMPLATE/training.yml: 対応するチェック項目を修正
- docs/admin-setup.md: 制約の詳細と、有効化するための2つの選択肢を記録

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Tivo0921
Tivo0921 merged commit c03f2a3 into main Aug 22, 2026
1 check passed
@Tivo0921
Tivo0921 deleted the docs/fix-branch-protection-claims branch August 22, 2026 12:49
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