Skip to content

fix: change shell execution from sh to bash for upload script#125

Merged
victorvhs017 merged 1 commit intomainfrom
fix/update-upload-workflow-to-use-bash
Feb 11, 2026
Merged

fix: change shell execution from sh to bash for upload script#125
victorvhs017 merged 1 commit intomainfrom
fix/update-upload-workflow-to-use-bash

Conversation

@victorvhs017
Copy link
Contributor

Description 📣

Updating the action to use bash so we can use the error handling on the upload script set -eo pipefail

Type ✨

  • Bug fix
  • New feature
  • Improvement
  • Breaking change
  • Documentation

- Updated the workflow to use bash instead of sh for executing the upload_to_cloudsmith.sh script, ensuring compatibility with bash-specific features.
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 11, 2026

Greptile Overview

Greptile Summary

This PR updates the GitHub Actions release workflow to execute the package upload script using bash instead of sh, enabling the script’s set -eo pipefail error-handling behavior to work as intended during releases.

The change is localized to .github/workflows/release_build_infisical_cli.yml and doesn’t modify the CLI code or release artifacts directly—only the shell used to run an existing upload step in CI.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk.
  • The diff is a small, targeted workflow adjustment (switching the shell to bash) intended to correctly enforce existing script error handling. No code paths, APIs, or security-sensitive logic in the application are changed.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/release_build_infisical_cli.yml Switched shell execution for an upload step from sh to bash to support set -eo pipefail; no functional issues found in the workflow change.

@victorvhs017 victorvhs017 merged commit 8fbee95 into main Feb 11, 2026
5 checks passed
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