Skip to content

Commit 9233acb

Browse files
lmeyerovclaude
andcommitted
refactor(ci): remove redundant lint-docs-rst job
Removed the separate lint-docs-rst job since RST validation is already integrated into the test-docs job via the Docker build process. The validation happens automatically in build-docs.sh before Sphinx runs, making a separate job unnecessary. This simplifies the CI workflow while maintaining the same validation coverage through the existing docs build pipeline. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent b9b6fbd commit 9233acb

1 file changed

Lines changed: 1 addition & 36 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -395,43 +395,8 @@ jobs:
395395
echo "✅ py.typed marker confirmed in wheel distribution"
396396
397397
398-
lint-docs-rst:
399-
needs: changes
400-
# Run if docs changed OR infrastructure changed OR manual/scheduled run
401-
if: ${{ needs.changes.outputs.docs == 'true' || needs.changes.outputs.infra == 'true' || github.event_name == 'workflow_dispatch' || github.event_name == 'schedule' }}
402-
runs-on: ubuntu-latest
403-
timeout-minutes: 2
404-
405-
steps:
406-
- name: Checkout repo
407-
uses: actions/checkout@v3
408-
409-
- name: Set up Python 3.10
410-
uses: actions/setup-python@v4
411-
with:
412-
python-version: '3.10'
413-
414-
- name: Install docs dependencies
415-
run: |
416-
python -m pip install --upgrade pip
417-
python -m pip install -e .[docs]
418-
419-
- name: Validate RST syntax
420-
run: |
421-
if [ -f .rstcheck.cfg ]; then
422-
echo "Using .rstcheck.cfg configuration"
423-
else
424-
echo "Warning: .rstcheck.cfg not found"
425-
fi
426-
# Run validation script if it exists, otherwise run rstcheck directly
427-
if [ -f bin/validate-docs.sh ]; then
428-
./bin/validate-docs.sh
429-
else
430-
find docs -name "*.rst" -exec rstcheck --config .rstcheck.cfg {} +
431-
fi
432-
433398
test-docs:
434-
needs: [changes, python-lint-types, lint-docs-rst]
399+
needs: [changes, python-lint-types]
435400
# Run if docs changed OR Python changed OR infrastructure changed OR manual/scheduled run
436401
if: ${{ needs.changes.outputs.docs == 'true' || needs.changes.outputs.python == 'true' || needs.changes.outputs.infra == 'true' || github.event_name == 'workflow_dispatch' || github.event_name == 'schedule' }}
437402
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)