Comment by notpushkin

Comment by notpushkin 2 days ago

3 replies

It would be a great rule for shellcheck, by the way.

  Line 6:
    curl -fsSL "${url}"
         ^-- SC8537 (warning): use long options instead (`--fail`, `--silent`, `--show-error`, `--location`).
yjftsjthsd-h 2 days ago

I would want it opt-in, because I use shellcheck on scripts that will be run on busybox or *BSD where there aren't long options

  • notpushkin 2 days ago

    Of course.

    • yjftsjthsd-h 2 days ago

      Oh, I didn't realize shellcheck already had optional checks (see `shellcheck --list-optional` for a list), so that was not obvious to me initially. Then yes, that'd be a good thing to have available.