ci: remove redundant docs workflow by cv · Pull Request #725 · NVIDIA/NemoClaw

cv · 2026-03-23T16:49:59Z

Summary

Remove docs.yaml workflow — docs-preview-pr.yaml already builds docs with -W (strict mode) on the same path triggers, making it a duplicate Sphinx build on every doc-touching PR.

Test plan

Verify docs-preview-pr.yaml still triggers and builds successfully on a doc change
Confirm no other workflows reference docs.yaml

🤖 Generated with Claude Code

Summary by CodeRabbit

Chores
- Removed the GitHub Actions workflow that automatically built and validated project documentation during pull requests.

docs-preview-pr.yaml already builds docs with -W (strict mode) on the same path triggers, making docs.yaml a duplicate build. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

coderabbitai · 2026-03-23T16:51:24Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 1000bd54-9f65-44a4-994f-7cbc68a6d2ec

📥 Commits

Reviewing files that changed from the base of the PR and between b6f45a6 and 65cef85.

📒 Files selected for processing (1)

.github/workflows/docs.yaml

💤 Files with no reviewable changes (1)

.github/workflows/docs.yaml

📝 Walkthrough

Walkthrough

A GitHub Actions workflow file for building and validating Sphinx documentation is removed. The workflow previously triggered on pull requests affecting documentation files and ran a job to build HTML documentation in strict mode using Python 3.11 and the uv package manager.

Changes

Cohort / File(s)	Summary
CI/CD Workflow Removal `.github/workflows/docs.yaml`	Deleted complete workflow definition that automated Sphinx documentation builds on pull requests, including environment setup, dependency synchronization, and strict HTML generation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A workflow takes its final bow,
Doc checks fade away somehow,
Simpler CI, no longer bound,
In quieter pipelines, peace is found!
✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'ci: remove redundant docs workflow' clearly and concisely summarizes the main change—removing a duplicate GitHub Actions workflow that was redundant with an existing one.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

docs-preview-pr.yaml already builds docs with -W (strict mode) on the same path triggers, making docs.yaml a duplicate build. Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

This reverts commit b18627f.

This reverts commit 36fa334. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix(ci): post docs preview comment with explicit PR number The workflow_run context has no github.event.pull_request.number, so rossjrw/pr-preview-action's internal sticky comment silently skips posting. Disable its built-in comment and add an explicit marocchino/sticky-pull-request-comment step that passes the PR number from the artifact metadata. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * test: test * fix: remove test * ci: revert "revert ci: remove redundant docs workflow (#725)" (#856) This reverts commit 36fa334. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: Miyoung Choi <miyoungc@nvidia.com>

This reverts commit b18627f.

* fix(ci): post docs preview comment with explicit PR number The workflow_run context has no github.event.pull_request.number, so rossjrw/pr-preview-action's internal sticky comment silently skips posting. Disable its built-in comment and add an explicit marocchino/sticky-pull-request-comment step that passes the PR number from the artifact metadata. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * test: test * fix: remove test * ci: revert "revert ci: remove redundant docs workflow (NVIDIA#725)" (NVIDIA#856) This reverts commit 36fa334. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: Miyoung Choi <miyoungc@nvidia.com>

This reverts commit b18627f.

* fix(ci): post docs preview comment with explicit PR number The workflow_run context has no github.event.pull_request.number, so rossjrw/pr-preview-action's internal sticky comment silently skips posting. Disable its built-in comment and add an explicit marocchino/sticky-pull-request-comment step that passes the PR number from the artifact metadata. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * test: test * fix: remove test * ci: revert "revert ci: remove redundant docs workflow (NVIDIA#725)" (NVIDIA#856) This reverts commit 36fa334. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: Miyoung Choi <miyoungc@nvidia.com>

docs-preview-pr.yaml already builds docs with -W (strict mode) on the same path triggers, making docs.yaml a duplicate build. Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

This reverts commit b18627f.

* fix(ci): post docs preview comment with explicit PR number The workflow_run context has no github.event.pull_request.number, so rossjrw/pr-preview-action's internal sticky comment silently skips posting. Disable its built-in comment and add an explicit marocchino/sticky-pull-request-comment step that passes the PR number from the artifact metadata. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * test: test * fix: remove test * ci: revert "revert ci: remove redundant docs workflow (NVIDIA#725)" (NVIDIA#856) This reverts commit 36fa334. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: Miyoung Choi <miyoungc@nvidia.com>

ci: remove redundant docs workflow

65cef85

docs-preview-pr.yaml already builds docs with -W (strict mode) on the same path triggers, making docs.yaml a duplicate build. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

cv requested a review from miyoungc March 23, 2026 18:11

jyaunches approved these changes Mar 23, 2026

View reviewed changes

cv requested review from miyoungc and removed request for miyoungc March 23, 2026 18:34

miyoungc approved these changes Mar 23, 2026

View reviewed changes

miyoungc merged commit b18627f into NVIDIA:main Mar 23, 2026
7 checks passed

cv deleted the ci/remove-duplicate-workflows branch March 23, 2026 19:04

miyoungc added a commit that referenced this pull request Mar 25, 2026

Revert "ci: remove redundant docs workflow (#725)"

8903e45

This reverts commit b18627f.

miyoungc mentioned this pull request Mar 25, 2026

fix: revert "ci: remove redundant docs workflow" #856

Merged

ericksoa pushed a commit that referenced this pull request Mar 25, 2026

Revert "ci: remove redundant docs workflow (#725)" (#856)

36fa334

This reverts commit b18627f.

cv added a commit that referenced this pull request Mar 25, 2026

ci: revert "revert ci: remove redundant docs workflow (#725)" (#856)

73b5ea0

This reverts commit 36fa334. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

temrjan pushed a commit to temrjan/NemoClaw that referenced this pull request Mar 25, 2026

Revert "ci: remove redundant docs workflow (NVIDIA#725)" (NVIDIA#856)

fc9210c

This reverts commit b18627f.

lakamsani pushed a commit to lakamsani/NemoClaw that referenced this pull request Apr 4, 2026

Revert "ci: remove redundant docs workflow (NVIDIA#725)" (NVIDIA#856)

30b942f

This reverts commit b18627f.

jacobtomlinson pushed a commit to jacobtomlinson/NemoClaw that referenced this pull request Apr 30, 2026

Revert "ci: remove redundant docs workflow (NVIDIA#725)" (NVIDIA#856)

7b39643

This reverts commit b18627f.

wscurran added the chore Build, CI, dependency, or tooling maintenance label Jun 8, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ci: remove redundant docs workflow#725

ci: remove redundant docs workflow#725
miyoungc merged 1 commit into
NVIDIA:mainfrom
cv:ci/remove-duplicate-workflows

cv commented Mar 23, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Mar 23, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Poem

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

cv commented Mar 23, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Mar 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Poem

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

cv commented Mar 23, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Mar 23, 2026 •

edited

Loading