Repository navigation
refactor README - #38
Merged
Merged
Conversation
susumutomita
enabled auto-merge (squash)
November 14, 2025 21:58
|
Caution Review failedThe pull request is closed. WalkthroughThe pull request removes the Contact section and footer from the README.md file, including references to GitHub Issues, Discussions, and the community attribution line. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
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. Comment |
PR Review: README.md リファクタリング概要このPRは README.md から連絡先セクションと「Built with ❤️」メッセージを削除するリファクタリングです。 ✅ 良い点
📋 観察事項1. コミットメッセージの形式現在のコミットメッセージ: CLAUDE.md:478-490 で定義されている Conventional Commits 形式に従うと、より詳細な説明が望ましいです: 2. PRの説明PR body が空です。CLAUDE.md:502-518 のガイドラインに従い、以下の情報を追加することを推奨します: ## 概要
README.md から CLAUDE.md と重複する連絡先セクションを削除し、ドキュメントを簡潔化
## 変更内容
- [x] 連絡先セクション削除（CLAUDE.md に既存のため）
- [x] フッターメッセージ削除
## 理由
ドキュメントの一元管理と重複削減のため🎯 推奨アクション（オプション）今後の PR のために以下を推奨します:
✅ 結論Approve 推奨
改善提案はあくまで今後のプロセス向上のためのもので、この PR 自体はマージ可能な状態です。 Review conducted following CLAUDE.md guidelines |
2 of 3 tasks
susumutomita
added a commit
that referenced
this pull request
Jun 4, 2026
…s red team (#1708) The Disruptions (red team) tab was empty because the problems submodule was pinned to de6a8f5, which predates the sample Battle getting a real red team. Bump the pin to a7ac11e (TenkaCloudChallenge main), which brings in: - #38 hello-world-battle: a real, explained red team — "frontend-down" disruption (ssm-run-command stops nginx; ADR-029 auto-revert after 600s). This is what makes the Disruptions tab non-empty so organizers can actually test the feature. - #39 microservice-migration-battle: red-team explanation in the description. - #40 problem "check engine" validator; #41 teaser ASCII-diagram fix. Also update apps/participant-portal/src/plugins/props-builder.test.ts: it used hello-world-battle as a "no disruptions" example, which is no longer true — switch the empty-case example to a challenge and pin the sample's new public disruption. To see it live, redeploy the problem-deploy backend (the event-handler Lambda bundles the catalog). Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit