Skip to content

Fix release workflow#30

Merged
fizikiukas merged 1 commit intomainfrom
fix/release-workflow
Dec 4, 2025
Merged

Fix release workflow#30
fizikiukas merged 1 commit intomainfrom
fix/release-workflow

Conversation

@fizikiukas
Copy link
Copy Markdown
Contributor

@fizikiukas fizikiukas commented Dec 4, 2025

Summary by CodeRabbit

  • Chores
    • Improved the package build and release process to ensure the package is properly built before being published.

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

@fizikiukas fizikiukas requested a review from a team as a code owner December 4, 2025 08:08
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 4, 2025

📝 Walkthrough

Walkthrough

A Build package step was added to the publish-npmjs job in the GitHub Actions workflow. This step executes npm build during the package publication process, positioned between the npm install and npm publish stages.

Changes

Cohort / File(s) Change Summary
CI/CD Workflow Configuration
.github/workflows/build-release.yaml
Added a Build package step in the publish-npmjs job to run npm build between npm install and npm publish

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single workflow file modification with straightforward step addition
  • No logic changes or file structure modifications
  • Configuration change following existing workflow patterns

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Fix release workflow' is vague and generic. While it relates to the changeset (modifying .github/workflows/build-release.yaml), it doesn't clearly convey the specific improvement: adding a Build package step to run npm build during the release process. Consider using a more specific title like 'Add npm build step to release workflow' to clearly communicate the primary change being made.
✅ Passed checks (2 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.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/release-workflow

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fbf580e and 7937429.

📒 Files selected for processing (1)
  • .github/workflows/build-release.yaml (1 hunks)
🔇 Additional comments (1)
.github/workflows/build-release.yaml (1)

28-29: > Likely an incorrect or invalid review comment.


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

@fizikiukas fizikiukas merged commit 9dab621 into main Dec 4, 2025
1 check passed
@fizikiukas fizikiukas deleted the fix/release-workflow branch December 4, 2025 08:13
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