Skip to content

Conversation

@pandeymangg
Copy link
Contributor

fixes the release action errors

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 1, 2025

@pandeymangg pandeymangg requested a review from Dhruwang October 1, 2025 07:31
@pandeymangg pandeymangg added this pull request to the merge queue Oct 1, 2025
Merged via the queue into main with commit ab71b45 Oct 1, 2025
8 of 9 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Oct 1, 2025

Walkthrough

The GitHub Actions workflow at .github/workflows/release.yml was updated to trigger on release events of type "released" instead of "published." In the Setup Node.js step, the configuration removed cache: "pnpm" and always-auth: true, eliminating explicit PNPM caching and the always-auth setting. No exported or public entity declarations were altered.

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title clearly summarizes the primary change of fixing release workflow errors and is concise enough to convey the main purpose of the pull request.
Description Check ✅ Passed The description accurately relates to the pull request by stating its purpose of fixing errors in the release action and aligns with the changes made in the workflow file.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

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

@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.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bce6a62 and 7356ab3.

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

on:
release:
types: [published]
types: [released]
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

Restore the release trigger to published.

Using types: [released] only fires when a prerelease is promoted to a full release. Regular releases published directly from draft (the common path) emit the published activity instead, so this workflow will stop running on new releases.(docs.github.com)

-    types: [released]
+    types: [published]
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
types: [released]
types: [published]
🤖 Prompt for AI Agents
In .github/workflows/release.yml around line 9 the workflow trigger uses types:
[released], which only fires when a prerelease is promoted; replace it with
types: [published] so the workflow runs for normal release publishes (i.e., set
types: [published] under the release trigger), verify YAML indentation/syntax
remains correct, and commit the change.

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.

3 participants