Skip to content

Conversation

@Shane32
Copy link
Owner

@Shane32 Shane32 commented Oct 5, 2025

Summary by CodeRabbit

  • Chores
    • Updated release CI to use the default Windows shell on Windows runners and align path syntax for package publishing.
    • Simplified the publish step configuration in build and release pipelines to improve reliability and consistency.
    • No user-facing changes.

@Shane32 Shane32 merged commit aeaf5e5 into master Oct 5, 2025
3 checks passed
@Shane32 Shane32 deleted the fix-ci-2 branch October 5, 2025 15:54
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 5, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Adjusts NuGet publish steps in two GitHub Actions workflows by removing explicit bash shells, letting them run under the default Windows shell, and changing the push path wildcard from out/* to out*.

Changes

Cohort / File(s) Summary of changes
CI workflows: publish step shell/path tweak
.github/workflows/wf-build-release-ci.yml, .github/workflows/wf-build-release.yml
Removed explicit bash shell; default Windows shell (pwsh) now used. Updated NuGet push path from out/* to out\* to match Windows-style wildcard escaping.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 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-2

📜 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 75720a5 and 3833005.

📒 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 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