fix: use the bash executable specified by PATH

TypedDevs / bashunit

A simple testing library for bash scripts. Test your bash scripts in the fastest and simplest way, discover the most modern bash testing library.

https://bashunit.typeddevs.com

MIT License

270 stars 21 forks source link

fix: use the bash executable specified by PATH #242

Closed apotterri closed 5 months ago

apotterri commented 5 months ago

📚 Description

Update the release script so the final bashunit script will use PATH to find bash. The version of bash in /bin on macOS is very old, and forcing tests to use it is quite limiting.

🔖 Changes

Change build.sh so it prepends #!/usr/bin/env bash to the final bashunit script.

✅ To-do list

[ ] I updated the CHANGELOG.md to reflect the new feature or fix
[ ] I updated the documentation to reflect the changes

apotterri commented 5 months ago

This seems like a workable approach, but I'm not sure it's best (especially because there don't appear to be any tests that use the final packaged bashunit script).

Happy to get any feedback on this change in general as well.

Chemaclass commented 5 months ago

Everything seems to be working fine. Thanks for the PR, @apotterri !

Screenshot 2024-01-13 at 17 07 24