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

shellcheck fixes #32

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

shellcheck fixes #32

wants to merge 12 commits into from

Commits on Nov 13, 2023

  1. SC2006: Use $(...) notation instead of legacy backticks

    @xavierba
    Xavier Bachelot authored and xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    4699007 View commit details
    Browse the repository at this point in the history

  2. SC2086: Double quote to prevent globbing and word splitting

    @xavierba
    Xavier Bachelot authored and xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    333ffa9 View commit details
    Browse the repository at this point in the history

  3. SC2003: drop use of expr

    @xavierba
    Xavier Bachelot authored and xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    c33a389 View commit details
    Browse the repository at this point in the history

  4. SC2116: useless use of echo

    @xavierba
    Xavier Bachelot authored and xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    08a804d View commit details
    Browse the repository at this point in the history

  5. SC2166: Prefer [ p ] && [ q ] as [ p -a q ] is not well defined

    @xavierba
    Xavier Bachelot authored and xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    7b5c754 View commit details
    Browse the repository at this point in the history

  6. SC2004: $/${} is unnecessary on arithmetic variables

    @xavierba
    Xavier Bachelot authored and xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    8ef8566 View commit details
    Browse the repository at this point in the history

  7. Disable SC2153 for MY_LDAP_ROOTPW

    @xavierba
    Xavier Bachelot authored and xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    0aa1002 View commit details
    Browse the repository at this point in the history

  8. SC2162: read without -r will mangle backslashes

    @xavierba
    Xavier Bachelot authored and xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    7e6ee9b View commit details
    Browse the repository at this point in the history

  9. SC2006: Use $(...) notation instead of legacy backticks

    @xavierba
    xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    7c66a07 View commit details
    Browse the repository at this point in the history

  10. SC2086: Double quote to prevent globbing and word splitting

    @xavierba
    xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    29a917a View commit details
    Browse the repository at this point in the history

  11. Workaround SC2028: echo may not expand escape sequences. Use printf

    @xavierba
    xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    8cf1912 View commit details
    Browse the repository at this point in the history

  12. More double quotes

    @xavierba
    xavierba committed Nov 13, 2023
    Configuration menu
    Copy the full SHA
    8e70190 View commit details
    Browse the repository at this point in the history