Skip to content

ci(workflows): normalize workflow display names#5381

Merged
cv merged 1 commit into
mainfrom
codex/rename-workflow-display-names
Jun 13, 2026
Merged

ci(workflows): normalize workflow display names#5381
cv merged 1 commit into
mainfrom
codex/rename-workflow-display-names

Conversation

@cv

@cv cv commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

Normalize GitHub Actions workflow display names so every workflow follows the repository's Category / Name naming structure. The NVSkills workflow keeps the required request-nvskills-ci.yml path and CODEOWNERS protection; investigation confirmed the central validator checks the workflow run path, not the display name.

Changes

  • Rename the Brev nightly workflow display name to E2E / Brev Nightly.
  • Rename the latest-tag promotion workflow display name to Release / Latest Tag.
  • Rename the NVSkills request workflow display name to Automation / Request NVSkills CI and adjust its source-template comment from Mirrors to Adapted from.

Type of Change

  • Code change (feature, bug fix, or refactor)
  • Code change with doc updates
  • Doc only (prose changes, no code sample modifications)
  • Doc only (includes code sample changes)

Verification

Docs writer check found no user-facing docs changes were needed.

  • npx prek run --all-files passes
  • npm test passes
  • Tests added or updated for new or changed behavior
  • No secrets, API keys, or credentials committed
  • Docs updated for user-facing behavior changes
  • npm run docs builds without warnings (doc changes only)
  • Doc pages follow the style guide (doc changes only)
  • New doc pages include SPDX header and frontmatter (new pages only)

Signed-off-by: Carlos Villela cvillela@nvidia.com

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflow display names for improved organization and clarity within the CI/CD pipeline infrastructure.

@cv cv self-assigned this Jun 13, 2026
@coderabbitai

coderabbitai Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: f2c5bc82-7275-440d-85c2-e18bc74ae353

📥 Commits

Reviewing files that changed from the base of the PR and between 1467e84 and 987e502.

📒 Files selected for processing (3)
  • .github/workflows/brev-nightly-e2e.yaml
  • .github/workflows/release-latest-tag.yaml
  • .github/workflows/request-nvskills-ci.yml

📝 Walkthrough

Walkthrough

This PR standardizes display names across three GitHub Actions workflows by adding category prefixes and consistent capitalization. No functional workflow logic is changed; only the visible names in the GitHub Actions UI are updated for improved organization and readability.

Changes

CI Workflow Naming

Layer / File(s) Summary
Standardize workflow display names
.github/workflows/brev-nightly-e2e.yaml, .github/workflows/release-latest-tag.yaml, .github/workflows/request-nvskills-ci.yml
Three CI workflows receive updated display names with consistent category prefixes: E2E, Release, and Automation. The request-nvskills-ci.yml comment header is also synchronized. No job logic, triggers, or concurrency settings are modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

area: ci, area: e2e

Poem

🐰 Workflows now wear tidy names,
With prefixes clean and the same,
E2E, Release, Automation true—
The UI is organized through and through! ✨

🚥 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 clearly and specifically summarizes the main change: normalizing workflow display names across three GitHub Actions workflows to follow a consistent 'Category / Name' pattern.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/rename-workflow-display-names

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

@github-actions

Copy link
Copy Markdown
Contributor

E2E Advisor Recommendation

Required E2E: None
Optional E2E: None

Workflow run

Full advisor summary

E2E Recommendation Advisor

Failed: Could not parse JSON from advisor output; see /home/runner/work/NemoClaw/NemoClaw/artifacts/e2e-advisor/e2e-advisor-raw-output.txt

@github-actions

Copy link
Copy Markdown
Contributor

Vitest E2E Scenario Recommendation

Required Vitest E2E scenarios: None
Optional Vitest E2E scenarios: None

Workflow run

Full Vitest E2E advisor summary

Vitest E2E Scenario Advisor

Failed: Could not parse JSON from advisor output; see /home/runner/work/NemoClaw/NemoClaw/artifacts/e2e-advisor/e2e-scenario-advisor-raw-output.txt

@github-actions

Copy link
Copy Markdown
Contributor

PR Review Advisor

Findings: 0 needs attention, 1 worth checking, 0 nice ideas
Top item: PR review advisor unavailable

Review findings

🛠️ Needs attention

  • None.

🔎 Worth checking

  • PR review advisor unavailable: The automated advisor could not complete: Could not parse JSON from PR review advisor output; see /home/runner/work/NemoClaw/NemoClaw/artifacts/pr-review-advisor/pr-review-advisor-raw-output.txt
    • Recommendation: Re-run the PR Review Advisor or perform a manual review.
    • Evidence: Could not parse JSON from PR review advisor output; see /home/runner/work/NemoClaw/NemoClaw/artifacts/pr-review-advisor/pr-review-advisor-raw-output.txt

🌱 Nice ideas

  • None.

Workflow run details

This is an automated advisory review. A human maintainer must make the final merge decision.

@cv cv merged commit 57b1f69 into main Jun 13, 2026
42 checks passed
@cv cv deleted the codex/rename-workflow-display-names branch June 13, 2026 07:53
@cv cv added the v0.0.65 Release target label Jun 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v0.0.65 Release target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant