feat: enhance Supabase CLI DevContainer feature documentation - #274
Closed
keito4 wants to merge 1 commit into
Closed
feat: enhance Supabase CLI DevContainer feature documentation#274keito4 wants to merge 1 commit into
keito4 wants to merge 1 commit into
Conversation
Supabase CLIのDevContainer Feature推奨設定を拡充: - データベースマイグレーション、型生成、ローカル開発環境の利点を明記 - supabase/config.toml存在時の条件付き推奨を追加 - Deno Runtimeとの併用パターンを推奨 - 公式ドキュメントへのリンクを追加 Closes #242 🤖 Generated with [Claude Code](https://claude.ai/code) Co-authored-by: keito4 <keito4@users.noreply.github.com>
|
Warning Rate limit exceeded@keito4 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 7 minutes and 16 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 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 |
keito4
added a commit
that referenced
this pull request
Jan 2, 2026
## Problem When PRs contain only documentation changes (e.g., `.md` files), the CI workflow is not triggered due to path filters. This causes check-ci-status to wait indefinitely for Quality Gate, eventually timing out after 15 minutes. ### Example: PR #274 - Changes: `.claude/devcontainer-recommendations.md` - CI workflow: Not triggered (path filters exclude `.md`) - check-ci-status: Waits for Quality Gate that never appears - Result: Times out after 900s, claude-review never runs ## Solution Added logic to detect when CI is skipped by path filters: ### Detection Logic 1. After 2 minutes (120s) of waiting 2. If Quality Gate hasn't appeared 3. Check if Detect Changes job exists 4. If both are missing → CI skipped by path filters 5. Set `ci_passed=true` to allow claude-review to proceed ### Behavior **Code changes** (CI runs): - Waits for Quality Gate completion - Validates all checks passed - Runs claude-review after CI success **Documentation-only changes** (CI skipped): - Detects no CI after 2 minutes - Allows claude-review to proceed immediately - No unnecessary 15-minute wait ### Status Messages - ✅ CI workflow skipped by path filters (no code changes detected) - Claude Code Review will proceed without waiting for CI ## Impact - PR #274 and similar doc-only PRs will get claude-review - No more 15-minute timeouts for non-code changes - Maintains CI validation for code changes 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
2 tasks
keito4
added a commit
that referenced
this pull request
Jan 2, 2026
## Problem When PRs contain only documentation changes (e.g., `.md` files), the CI workflow is not triggered due to path filters. This causes check-ci-status to wait indefinitely for Quality Gate, eventually timing out after 15 minutes. ### Example: PR #274 - Changes: `.claude/devcontainer-recommendations.md` - CI workflow: Not triggered (path filters exclude `.md`) - check-ci-status: Waits for Quality Gate that never appears - Result: Times out after 900s, claude-review never runs ## Solution Added logic to detect when CI is skipped by path filters: ### Detection Logic 1. After 2 minutes (120s) of waiting 2. If Quality Gate hasn't appeared 3. Check if Detect Changes job exists 4. If both are missing → CI skipped by path filters 5. Set `ci_passed=true` to allow claude-review to proceed ### Behavior **Code changes** (CI runs): - Waits for Quality Gate completion - Validates all checks passed - Runs claude-review after CI success **Documentation-only changes** (CI skipped): - Detects no CI after 2 minutes - Allows claude-review to proceed immediately - No unnecessary 15-minute wait ### Status Messages - ✅ CI workflow skipped by path filters (no code changes detected) - Claude Code Review will proceed without waiting for CI ## Impact - PR #274 and similar doc-only PRs will get claude-review - No more 15-minute timeouts for non-code changes - Maintains CI validation for code changes 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
Owner
Author
|
Closing to recreate with updated claude-code-review workflow that handles path filter skips (PR #275) |
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
Supabase CLIのDevContainer Feature推奨設定を拡充しました。
Changes
supabase/config.toml存在時の条件付き推奨を追加Benefits
Test Plan
Closes #242
🤖 Generated with Claude Code