Skip to content

ci: Update release-docs workflow to use FW-CI-templates v0.72.0#1965

Merged
chtruong814 merged 1 commit intomainfrom
chtruong/update-release-docs-latest
Feb 17, 2026
Merged

ci: Update release-docs workflow to use FW-CI-templates v0.72.0#1965
chtruong814 merged 1 commit intomainfrom
chtruong/update-release-docs-latest

Conversation

@chtruong814
Copy link
Contributor

@chtruong814 chtruong814 commented Feb 15, 2026

What does this PR do ?

ci: Update release-docs workflow to use FW-CI-templates v0.72.0

The latest tag in some repos was whatever was published from main branch. However, we will now designate "latest" as the latest stable version. A follow-up change will include publshing docs as part of the release repo workflow. And we will also begin to publish "nightly" docs from the main branch.

Issues

List issues that this PR closes (syntax):

Usage

  • You can potentially add a usage example below
# Add a code snippet demonstrating how to use this

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you run the unit tests and functional tests locally? Visit our Testing Guide for how to run tests
  • Did you add or update any necessary documentation? Visit our Document Development Guide for how to write, build and test the docs.

Additional Information

  • ...

Summary by CodeRabbit

  • Chores
    • Added new workflow input to control whether releases are published as the latest version (enabled by default)
    • Updated CI/CD infrastructure dependencies

Signed-off-by: Charlie Truong <chtruong@nvidia.com>
@chtruong814 chtruong814 requested a review from a team as a code owner February 15, 2026 23:57
@github-actions github-actions bot added the CI Relating to CI label Feb 15, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 16, 2026

📝 Walkthrough

Walkthrough

This PR updates the release-docs workflow by introducing a new optional workflow input publish-as-latest (defaulting to true), bumping the FW-CI-templates reference from v0.71.0 to v0.72.0, and dynamically parameterizing the overwrite-latest-on-tag setting based on the new input.

Changes

Cohort / File(s) Summary
Release Docs Workflow Configuration
.github/workflows/release-docs.yml
Added new workflow input publish-as-latest (boolean, default true); upgraded FW-CI-templates reference to v0.72.0; changed overwrite-latest-on-tag from static false to dynamic input-driven value.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

CI, CI:docs

Suggested reviewers

  • ko3n1g
  • terrykong
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the primary change—updating the release-docs workflow to use FW-CI-templates v0.72.0, which is the main change evident in the raw summary.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main
Test Results For Major Changes ✅ Passed PR contains only minor CI/CD workflow configuration changes to documentation publishing, not affecting runtime behavior, numerics, convergence, or performance.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chtruong/update-release-docs-latest

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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 and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
.github/workflows/release-docs.yml (1)

52-57: ⚠️ Potential issue | 🟡 Minor

Stale inline comment — no longer reflects the new behavior.

These comments still describe the old behavior where running on main publishes to the "latest" directory. With overwrite-latest-on-tag now driven by the publish-as-latest input, "latest" is controlled explicitly and is no longer tied to being on main. Please update or remove the comment to avoid confusion.

✏️ Suggested comment update
       - uses: ./FW-CI-templates/.github/actions/publish-docs
-        # This workflow runs either on main, or on a version tag. Any other git ref will lead
-        # to an error.
-        # If its on main, it will publish to "latest" directory in Akamai.
-        # If its on a versioned tag, it will extract the version number from the tag (strip `v` prefix)
-        # and publish to the versioned directory in Akamai.
+        # This workflow runs either on main or on a version tag. Any other git ref will lead
+        # to an error.
+        # On a versioned tag, it extracts the version number (strips `v` prefix) and publishes
+        # to the versioned directory in Akamai. If publish-as-latest is true, it also overwrites
+        # the "latest" directory.

@chtruong814 chtruong814 added the CI:docs Run doctest label Feb 17, 2026
@chtruong814 chtruong814 enabled auto-merge (squash) February 17, 2026 17:23
@chtruong814 chtruong814 merged commit 0955329 into main Feb 17, 2026
41 of 42 checks passed
@chtruong814 chtruong814 deleted the chtruong/update-release-docs-latest branch February 17, 2026 17:33
yuanhangsu1986 pushed a commit to yuanhangsu1986/RL-Nemontron-Edge-Omni that referenced this pull request Feb 21, 2026
…IA-NeMo#1965)

Signed-off-by: Charlie Truong <chtruong@nvidia.com>
Signed-off-by: yuanhangs <yuanhangs@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:docs Run doctest CI Relating to CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants