Skip to content

[master] use old release process#3335

Closed
openshift-cherrypick-robot wants to merge 1 commit intoopenshift-assisted:masterfrom
openshift-cherrypick-robot:cherry-pick-3243-to-master
Closed

[master] use old release process#3335
openshift-cherrypick-robot wants to merge 1 commit intoopenshift-assisted:masterfrom
openshift-cherrypick-robot:cherry-pick-3243-to-master

Conversation

@openshift-cherrypick-robot
Copy link
Contributor

@openshift-cherrypick-robot openshift-cherrypick-robot commented Dec 18, 2025

This is an automated cherry-pick of #3243

/assign jgyselov

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflow configuration to improve build and release automation reliability and security posture through streamlined dependencies and authentication handling.

✏️ Tip: You can customize this high-level summary in your review settings.

@openshift-ci
Copy link

openshift-ci bot commented Dec 18, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: openshift-cherrypick-robot
Once this PR has been reviewed and has the lgtm label, please assign jgyselov for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@coderabbitai
Copy link

coderabbitai bot commented Dec 18, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request modifies the GitHub Actions release workflow to downgrade action versions (checkout v4→v3, setup-node v4→v3), add a git configuration step, remove Yarn registry configuration, and switch from hardcoded registry settings to NPM_AUTH_TOKEN-based authentication for package publication.

Changes

Cohort / File(s) Change Summary
Release Workflow
​.github/workflows/release.yaml
Downgrades checkout and setup-node actions; adds git user config; removes Yarn registry configuration; replaces hardcoded npm registry/auth with NPM_AUTH_TOKEN environment variable; simplifies npm publish command

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Single file modification with configuration and action version changes
  • Straightforward substitution of authentication mechanism (hardcoded → token-based)
  • Requires verification of action version compatibility and npm authentication flow correctness

Possibly related PRs

Suggested labels

lgtm, approved, size/S

Suggested reviewers

  • jgyselov
  • ammont82

Poem

🐰 A workflow refined with care,
Authentication token in the air,
Versions downward, gitconfig set,
Cleaner release pipeline yet!
Yarn's old way now has rest,
NPM_AUTH_TOKEN does best. ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 60b2e0f and 63a72c1.

📒 Files selected for processing (1)
  • .github/workflows/release.yaml (3 hunks)

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.

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Dec 18, 2025
@jgyselov jgyselov closed this Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants