Skip to content

Fix YAML syntax error in build-venv-probe.yml - #19

Merged
Wewoc merged 1 commit into
mainfrom
claude/magical-ritchie-807x8r
Sep 30, 2026
Merged

Wewoc merged 1 commit into
mainfrom
claude/magical-ritchie-807x8r

Conversation

@Wewoc

@Wewoc Wewoc commented Sep 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • Root cause of the workflow_dispatch dispatch failures found: the step name Check D: drive exists and is writable has a colon followed by a space (D: drive) inside an unquoted YAML scalar — invalid YAML, looks like a nested mapping key.
  • This silently broke the whole file: GitHub couldn't parse it, so workflow_dispatch was never actually registered despite list_workflows showing it as active (with a path-based fallback name instead of the declared name:), and every push produced a synthetic 0-job "failure" run instead of actually running anything.
  • Fixed by quoting the step name.

Test plan

  • python3 -c "import yaml; yaml.safe_load(...)" confirms valid YAML now
  • Trigger via workflow_dispatch after merge, confirm it actually runs

🤖 Generated with Claude Code

https://claude.ai/code/session_01JYLx9STTgfo9KXLC2Gog3g


Generated by Claude Code

The step name "Check D: drive exists and is writable" has a colon
followed by a space ("D: drive") inside an unquoted scalar — invalid
YAML (looks like a nested mapping key). This silently broke the whole
workflow file: GitHub couldn't parse it, so workflow_dispatch was
never actually registered (despite list_workflows showing it as
"active" with a path-based fallback name instead of the declared
name), and every push produced a synthetic 0-job "failure" run instead
of running anything. Quoted the value to fix it.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JYLx9STTgfo9KXLC2Gog3g
@Wewoc
Wewoc merged commit b942a55 into main Sep 30, 2026
1 check passed
@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: afc72a94-34ea-4575-abb2-54583bf15730

📥 Commits

Reviewing files that changed from the base of the PR and between e97c124 and 2e15b02.

📒 Files selected for processing (1)
  • .github/workflows/build-venv-probe.yml
 ________________________
< Tree-sitter is my GPS. >
 ------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

2 participants