#3019 PR merged
: issue: #3018: fix sesam default exlude pathes¶
Labels: bug
, fixed / solved / done
, external tool
abbbi opened issue at 2023-06-26 12:37:¶
fix for issue #3018
jsmeix commented at 2023-06-26 12:55:¶
@rear/contributors
could you review this pull request (if time permits)?
jsmeix commented at 2023-06-27 09:28:¶
Those 'Code scanning / Shellcheck (reported by Codacy)
are from Codacy, cf.
https://github.com/rear/rear/commit/3cb64f1262eac953b78eec515b1ca561aa1c5b6b
See also
https://github.com/rear/rear/pull/2991#issuecomment-1573522079
and the subsequent comments, in particular see
https://github.com/rear/rear/pull/2991#issuecomment-1573690998
... it seems that Codacy simply calls plain 'shellcheck' ...
and
https://github.com/rear/rear/pull/2991#issuecomment-1573799743
jsmeix commented at 2023-06-28 07:29:¶
@rear/contributors
I would like to merge it today afternoon
unless objections appear.
FYI regarding ShellCheck:
Our own "Differential ShellCheck" (which is deliberately set up
exactly as currently needed in ReaR) does not report issues.
schlomo commented at 2023-06-28 07:37:¶
Yes, please merge. Can we disable those extra shellcheck errors from Codacy?
[Export of Github issue for rear/rear.]