Skip to content

fix: install script bugs and format#296

Open
RemoteRabbit wants to merge 7 commits intodatabricks-industry-solutions:mainfrom
RemoteRabbit:fix/install-script-bugs
Open

fix: install script bugs and format#296
RemoteRabbit wants to merge 7 commits intodatabricks-industry-solutions:mainfrom
RemoteRabbit:fix/install-script-bugs

Conversation

@RemoteRabbit
Copy link
Contributor

Summary

Cleanup and format the install script. Fix the shebang and also some issues with quotes.

🐛 Bug Fixes

  • fix: exit if on failure for the cd command [cc69b6d]
  • fix: use double quotes on version to prevent globbing [14ce974]
  • fix: use proper shebang [d5ca494]

🔧 Maintenance

📁 File Changes

Root (+10/-9 lines)

  • install.sh (+10/-9) 📝

Changes: 1 files, +10 insertions, -9 deletions
Commits: 4
Branch: fix/install-script-bugs
Base: origin/main

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have reviewed the CONTRIBUTING and VERSIONING documentation.
  • My code follows the code style of this project.
  • I have verified, added or updated the tests to cover my changes (if applicable).
  • I have verified that my changes do not introduce any breaking changes on all the supported clouds (AWS, Azure, GCP).
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation (if applicable).

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@RemoteRabbit RemoteRabbit changed the title Fix/install script bugs fix: install script bugs and format Feb 17, 2026
@RemoteRabbit RemoteRabbit force-pushed the fix/install-script-bugs branch from 6071bee to b50e603 Compare February 18, 2026 01:26
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