Skip to content

ci(fix): code freeze workflow#2177

Merged
ko3n1g merged 3 commits intomainfrom
ko3n1g/ci/code-freeze-workflow
Feb 2, 2026
Merged

ci(fix): code freeze workflow#2177
ko3n1g merged 3 commits intomainfrom
ko3n1g/ci/code-freeze-workflow

Conversation

@ko3n1g
Copy link
Copy Markdown
Contributor

@ko3n1g ko3n1g commented Feb 2, 2026

What does this PR do ?

Add a one line overview of what this PR aims to accomplish.

Changelog

  • Add specific line by line info of high level changes in this PR.

GitHub Actions CI

See the CI sectionin the Contributing doc for how to trigger the CI. A Nvidia developer will need to approve and trigger the CI for external contributors.

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 add or update any necessary documentation?
  • Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
    • Reviewer: Does the PR have correct import guards for all optional libraries?

If you haven't finished some of the above items you can still open "Draft" PR.

Additional Information

  • Related to # (issue)

Summary by CodeRabbit

  • Chores

    • Updated GitHub Actions workflows with enhanced credential management for improved security in automated release processes.
    • Improved branch synchronization procedures in release workflows to ensure latest remote state is properly synchronized before processing.
  • Documentation

    • Removed duplicate version entry from the documentation versions index for better data consistency.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Feb 2, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@ko3n1g ko3n1g force-pushed the ko3n1g/ci/code-freeze-workflow branch from 6eb8094 to 0357193 Compare February 2, 2026 13:07
Signed-off-by: oliver könig <okoenig@nvidia.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
@ko3n1g ko3n1g force-pushed the ko3n1g/ci/code-freeze-workflow branch from 0357193 to 2a45d51 Compare February 2, 2026 13:08
@ko3n1g ko3n1g marked this pull request as ready for review February 2, 2026 13:09
@ko3n1g ko3n1g requested a review from a team as a code owner February 2, 2026 13:09
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 2, 2026

📝 Walkthrough

Walkthrough

Updated GitHub Actions workflow to use a Personal Access Token for code-freeze operations and added git fetch before branch checkout. Removed duplicate version entry from docs/versions1.json file.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/release-freeze.yml
Added PAT configuration to code-freeze job and included git fetch origin before branch checkout in the "Switch to release branch" step to ensure latest remote state.
Documentation Versioning
docs/versions1.json
Removed duplicate version entry for "0.3.0" from the versions array.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'ci(fix): code freeze workflow' is partially related to the main changes but is overly broad and doesn't clearly specify what was fixed in the workflow. Make the title more specific about what was fixed, such as 'ci(fix): add PAT support to code-freeze workflow' to better convey the primary change.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Test Results For Major Changes ✅ Passed Changes are minor: GitHub Actions workflow PAT update and documentation metadata cleanup; neither affects core functionality.

✏️ 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 ko3n1g/ci/code-freeze-workflow

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.

@ko3n1g ko3n1g merged commit 549e3cb into main Feb 2, 2026
49 checks passed
@ko3n1g ko3n1g deleted the ko3n1g/ci/code-freeze-workflow branch February 2, 2026 14:52
yaoyu-33 pushed a commit that referenced this pull request Feb 3, 2026
Signed-off-by: oliver könig <okoenig@nvidia.com>
@coderabbitai coderabbitai bot mentioned this pull request Feb 23, 2026
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants