Skip to content

Conversation

@Shane32
Copy link
Owner

@Shane32 Shane32 commented Oct 5, 2025

Summary by CodeRabbit

  • Chores
    • Updated release workflows to explicitly use bash for the package publishing step, improving consistency and reliability of releases.
    • No user-facing functionality changes.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 5, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Explicitly sets shell: bash for the "Publish packages to Github packages" step in two GitHub Actions workflows. No other changes or public API impacts.

Changes

Cohort / File(s) Summary of Changes
CI Workflows
.github/workflows/wf-build-release-ci.yml, .github/workflows/wf-build-release.yml
Added shell: bash to the "Publish packages to Github packages" step to enforce Bash as the execution shell.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • gfoidl
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-ci-again

📜 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 c68ebce and 81b6375.

📒 Files selected for processing (2)
  • .github/workflows/wf-build-release-ci.yml (1 hunks)
  • .github/workflows/wf-build-release.yml (1 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.

@Shane32 Shane32 merged commit 75720a5 into master Oct 5, 2025
3 of 4 checks passed
@Shane32 Shane32 deleted the fix-ci-again branch October 5, 2025 15:32
@Shane32
Copy link
Owner Author

Shane32 commented Oct 5, 2025

It still doesn't work 👎

image

@Shane32 Shane32 added the ci Changes to ci/cd scripts or build infrastructure label Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes to ci/cd scripts or build infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants