Skip to content

Conversation

@bradegler
Copy link
Contributor

Linting changes introduced in #158 changed the shell on the script from the universally available /bin/sh to /usr/bin/env/bash which does not exist.

This change points the script to /bin/bash which should exist in GHA.

@bradegler bradegler requested a review from a team as a code owner April 28, 2025 19:20
@bradegler bradegler requested review from ruizhg and sqin2019 April 28, 2025 19:20
@bradegler bradegler enabled auto-merge (squash) April 28, 2025 19:21
@bradegler bradegler merged commit ab407dd into main Apr 28, 2025
1 check passed
@bradegler bradegler deleted the bradegler/fix_gen_version_check branch April 28, 2025 19:21
@sethvargo
Copy link
Contributor

Sorry, this should have been #!/usr/bin/env bash (NOT #!/usr/bin/env/bash). That sources bash from the environment instead of relying on a hardcoded path. I probably fat-fingered the wrong one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants