Skip to content

Commit 0d44bc6

Browse files
committed
Run markdown check inside dev container with full deps
Use the tests.yml workflow to run md-babel-py inside the dimos dev container instead of the standalone md-babel-py Docker image. This ensures all project dependencies (curl, git, python packages, etc.) are available when executing code blocks. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Former-commit-id: 386f736
1 parent ea735d9 commit 0d44bc6

1 file changed

Lines changed: 11 additions & 8 deletions

File tree

.github/workflows/docker.yml

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -217,14 +217,17 @@ jobs:
217217
dev-image: ros-dev:${{ (needs.check-changes.outputs.python == 'true' || needs.check-changes.outputs.dev == 'true' || needs.check-changes.outputs.ros == 'true') && needs.ros-dev.result == 'success' && needs.check-changes.outputs.branch-tag || 'dev' }}
218218

219219
check-markdown:
220-
needs: [check-changes]
221-
runs-on: [self-hosted, Linux]
222-
steps:
223-
- uses: actions/checkout@v4
224-
- uses: leshy/md-babel-py@main
225-
with:
226-
files: 'README.md docs/*.md'
227-
fail-on-change: true
220+
needs: [check-changes, dev]
221+
if: always()
222+
uses: ./.github/workflows/tests.yml
223+
with:
224+
should-run: ${{
225+
needs.check-changes.result == 'success' &&
226+
((needs.dev.result == 'success') ||
227+
(needs.dev.result == 'skipped'))
228+
}}
229+
cmd: "pip install md-babel-py && md-babel-py run README.md && md-babel-py run docs/*.md && git diff --exit-code"
230+
dev-image: dev:${{ (needs.check-changes.outputs.python == 'true' || needs.check-changes.outputs.dev == 'true') && needs.dev.result == 'success' && needs.check-changes.outputs.branch-tag || 'dev' }}
228231

229232
# Run module tests directly to avoid pytest forking issues
230233
# run-module-tests:

0 commit comments

Comments
 (0)