style(review-swarm): align the RELAY_WORKSPACE_KEY presence check - #236
Merged
Conversation
Whitespace only. I hand-resolved a conflict between #226, #232 and #235 an hour ago and carried the surrounding block's indentation onto the line I lifted out of it, so this check sat two spaces deeper than the two it belongs with. Shell does not care; a reader does, and misaligned indentation from a merge is the first thing that makes someone wonder what else the merge got wrong. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FtQSAcGDta5VH9xiZFT4sR
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 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 |
kjgbot
pushed a commit
that referenced
this pull request
Sep 8, 2026
… slip Ran #230's gate against the six merges (2 of 3 checks skip in flows, worth noting), then bash -n on every run: block since three PRs edited one shell block. 7/0. Caught and fixed a two-space misalignment from my conflict resolution in #236. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FtQSAcGDta5VH9xiZFT4sR
Review swarm: maintainabilityNo fresh transcript was produced for run |
Review swarm: historyNo fresh transcript was produced for run |
Review swarm: structureNo fresh transcript was produced for run |
Review swarm: FAILED
Cloud run: |
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.
Whitespace only, one line.
Resolving the three-way conflict between #226, #232 and #235 by hand, I lifted
test -n "$RELAY_WORKSPACE_KEY"out of the branch side and carried the surrounding block's indentation with it, so it sat two spaces deeper than the two presence checks it belongs with.Shell does not care. A reader does — misaligned indentation arriving from a merge is the first thing that makes someone wonder what else that merge got wrong, and three PRs edited this block in the last hour.
Verified: YAML parses, the step still passes
bash -n, and all threetest -nlines now share one indent level.🤖 Generated with Claude Code
https://claude.ai/code/session_01FtQSAcGDta5VH9xiZFT4sR