[PR #16] [MERGED] 🔧 refactor: Use #!/usr/bin/env bash in all shell scripts #19

New Issue

saavagebueno · 2025-11-20T04:14:19-05:00

saavagebueno commented

2025-11-20 04:14:19 -05:00

📋 Pull Request Information

Original PR: https://github.com/bigbeartechworld/big-bear-scripts/pull/16
Author: @dragonfire1119
Created: 10/23/2024
Status: ✅ Merged
Merged: 10/23/2024
Merged by: @dragonfire1119

Base: master ← Head: Make-shebangs-portable

📝 Commits (1)

321eba1 🔧 refactor: Use #!/usr/bin/env bash in all shell scripts

📊 Changes

61 files changed (+61 additions, -61 deletions)

View changed files

📝 batch-docker-volume-delete/run.sh (+1 -1)
📝 calculate-php-pool/run.sh (+1 -1)
📝 casaos-add-lan-ip-to-nextcloud-config/run.sh (+1 -1)
📝 casaos-add-pm-to-nextcloud/run.sh (+1 -1)
📝 casaos-appstore-edge/run.sh (+1 -1)
📝 casaos-appstore-fix/run.sh (+1 -1)
📝 casaos-check-if-blocked-by-ufw/run.sh (+1 -1)
📝 casaos-docker-compose-update/run.sh (+1 -1)
📝 casaos-generate-financial-freedom-app-key/run.sh (+1 -1)
📝 casaos-immich-vectors-upgrade/run.sh (+1 -1)
📝 change-casaos-web-ui-port/run.sh (+1 -1)
📝 check-for-docker-port/run.sh (+1 -1)
📝 check-if-docker-is-running-on-snap/run.sh (+1 -1)
📝 copy-codex-docs-config/run.sh (+1 -1)
📝 delete-docker-volumes-by-keyword/run.sh (+1 -1)
📝 disable-dns-service/disable_dns_service.sh (+1 -1)
📝 disk-usage-monitor/run.sh (+1 -1)
📝 docker-cleanup/run.sh (+1 -1)
📝 docker-exec-helper/run.sh (+1 -1)
📝 download-zim-files/run.sh (+1 -1)

...and 41 more files

📄 Description

This pull request refactors all the shell scripts in the project to use #!/usr/bin/env bash instead of #!/bin/bash as the shebang line. This change ensures that the scripts will run with the system's default Bash interpreter, even if it is not located at the standard /bin/bash path.

The main benefit of this change is improved compatibility across different systems, as the scripts will now work correctly regardless of the location of the Bash executable.

Summary by CodeRabbit

New Features
- Enhanced user prompts and logging throughout various scripts for improved interaction and clarity.
- Added functionality to handle configuration files and default values in several scripts.
- Introduced new checks for existing files and user confirmations before critical operations.
Bug Fixes
- Improved error handling and messaging for user feedback across multiple scripts.
Chores
- Updated shebang lines for better portability across different environments.
- Refined comments and documentation for clarity and maintainability.
Style
- Improved formatting and structure of output messages for better user experience.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/bigbeartechworld/big-bear-scripts/pull/16 **Author:** [@dragonfire1119](https://github.com/dragonfire1119) **Created:** 10/23/2024 **Status:** ✅ Merged **Merged:** 10/23/2024 **Merged by:** [@dragonfire1119](https://github.com/dragonfire1119) **Base:** `master` ← **Head:** `Make-shebangs-portable` --- ### 📝 Commits (1) - [`321eba1`](https://github.com/bigbeartechworld/big-bear-scripts/commit/321eba1a43c1044af8ba624893df6796f2e56332) 🔧 refactor: Use #!/usr/bin/env bash in all shell scripts ### 📊 Changes **61 files changed** (+61 additions, -61 deletions) <details> <summary>View changed files</summary> 📝 `batch-docker-volume-delete/run.sh` (+1 -1) 📝 `calculate-php-pool/run.sh` (+1 -1) 📝 `casaos-add-lan-ip-to-nextcloud-config/run.sh` (+1 -1) 📝 `casaos-add-pm-to-nextcloud/run.sh` (+1 -1) 📝 `casaos-appstore-edge/run.sh` (+1 -1) 📝 `casaos-appstore-fix/run.sh` (+1 -1) 📝 `casaos-check-if-blocked-by-ufw/run.sh` (+1 -1) 📝 `casaos-docker-compose-update/run.sh` (+1 -1) 📝 `casaos-generate-financial-freedom-app-key/run.sh` (+1 -1) 📝 `casaos-immich-vectors-upgrade/run.sh` (+1 -1) 📝 `change-casaos-web-ui-port/run.sh` (+1 -1) 📝 `check-for-docker-port/run.sh` (+1 -1) 📝 `check-if-docker-is-running-on-snap/run.sh` (+1 -1) 📝 `copy-codex-docs-config/run.sh` (+1 -1) 📝 `delete-docker-volumes-by-keyword/run.sh` (+1 -1) 📝 `disable-dns-service/disable_dns_service.sh` (+1 -1) 📝 `disk-usage-monitor/run.sh` (+1 -1) 📝 `docker-cleanup/run.sh` (+1 -1) 📝 `docker-exec-helper/run.sh` (+1 -1) 📝 `download-zim-files/run.sh` (+1 -1) _...and 41 more files_ </details> ### 📄 Description This pull request refactors all the shell scripts in the project to use `#!/usr/bin/env bash` instead of `#!/bin/bash` as the shebang line. This change ensures that the scripts will run with the system's default Bash interpreter, even if it is not located at the standard `/bin/bash` path. The main benefit of this change is improved compatibility across different systems, as the scripts will now work correctly regardless of the location of the Bash executable.  ## Summary by CodeRabbit - **New Features** - Enhanced user prompts and logging throughout various scripts for improved interaction and clarity. - Added functionality to handle configuration files and default values in several scripts. - Introduced new checks for existing files and user confirmations before critical operations. - **Bug Fixes** - Improved error handling and messaging for user feedback across multiple scripts. - **Chores** - Updated shebang lines for better portability across different environments. - Refined comments and documentation for clarity and maintainability. - **Style** - Improved formatting and structure of output messages for better user experience.  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

saavagebueno added the pull-request label 2025-11-20 04:14:19 -05:00

saavagebueno closed this issue

2025-11-20 04:14:20 -05:00

saavagebueno referenced this issue

2025-11-20 04:14:20 -05:00

[PR #19] [MERGED] 🔧 feat(casaos-toolkit): Add CasaOS toolkit scripts #23

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/big-bear-scripts#19