script: Disable SC1004 check in contrib/devtools/gen-bitcoin-conf.sh #25238

pull kristapsk wants to merge 1 commits into bitcoin:master from kristapsk:shellcheck-disable=SC1004 changing 1 files +1 −0
  1. kristapsk commented at 4:35 pm on May 29, 2022: contributor

    Otherwise ./test/lint/lint-all.py gives me error:

    0In contrib/devtools/gen-bitcoin-conf.sh line 58:
    1    | sed 's,\(^[[:upper:]].*\)\:$,\
    2                                   ^-- SC1004: This backslash+linefeed is literal. Break outside single quotes if you just want to break the line.
    3
    4For more information:
    5  https://www.shellcheck.net/wiki/SC1004 -- This backslash+linefeed is litera...
    6^---- failure generated from lint-shell.py
    
  2. Disable SC1004 check 55f0b40419
  3. hebasto commented at 4:41 pm on May 29, 2022: member
    If your shellcheck differs from v0.8.0 it could produce errors.
  4. kristapsk commented at 4:43 pm on May 29, 2022: contributor
    Just checked, I’m having v0.7.1. Will try to upgrade and test again.
  5. DrahtBot added the label Scripts and tools on May 29, 2022
  6. kristapsk commented at 6:34 pm on May 29, 2022: contributor
    After upgrading shellcheck to v0.8.0, linter error disappears, so closing.
  7. kristapsk closed this on May 29, 2022

  8. kristapsk deleted the branch on May 29, 2022
  9. fanquake locked this on May 31, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-30 15:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me