Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Modernize bash script #114

Merged
merged 3 commits into from
Jul 19, 2022
Merged

Modernize bash script #114

merged 3 commits into from
Jul 19, 2022

Commits on Jul 19, 2022

  1. Better structure for export file logic 

    We now either output stuff to the shell or output to the EXPORT_FILE.
    @svenstaro
    svenstaro committed Jul 19, 2022
    Configuration menu
    Copy the full SHA
    6a7d123 View commit details
    Browse the repository at this point in the history

  2. Make variable usage safer 

    Before we sometimes had unset variables in the script but it help to be stricter about this
so that this doesn't happen anymore in the future.
    @svenstaro
    svenstaro committed Jul 19, 2022
    Configuration menu
    Copy the full SHA
    aeb6099 View commit details
    Browse the repository at this point in the history

  3. Always use bash builtin conditional characters

    @svenstaro
    svenstaro committed Jul 19, 2022
    Configuration menu
    Copy the full SHA
    fe2fc24 View commit details
    Browse the repository at this point in the history