improve error handling 3 #25
pr-review.yml
on: pull_request
shellcheck grml-debootstrap
11s
shellcheck test scripts
7s
Annotations
10 errors and 10 warnings
shellcheck grml-debootstrap
reviewdog: Too many results (annotations) in diff.
You may miss some annotations due to GitHub limitation for annotation created by logging command.
Please check GitHub Actions log console to see all results.
Limitation:
- 10 warning annotations and 10 error annotations per step
- 50 annotations per job (sum of annotations from all the steps)
- 50 annotations per run (separate from the job annotations, these annotations aren't created by users)
Source: https://github.community/t5/GitHub-Actions/Maximum-number-of-annotations-that-can-be-created-using-GitHub/m-p/39085
|
shellcheck grml-debootstrap:
grml-debootstrap#L795
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:795:- RELEASE="$(dialog --stdout --title "${PN}" --default-item $DEFAULT_RELEASE --menu \
grml-debootstrap:795:+ RELEASE="$(dialog --stdout --title "${PN}" --default-item "$DEFAULT_RELEASE" --menu \
|
shellcheck grml-debootstrap:
grml-debootstrap#L847
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:847:- CHOOSE_MIRROR=$(dialog --stdout --title "$PN" --default-item $DEFAULT_MIRROR \
grml-debootstrap:847:+ CHOOSE_MIRROR=$(dialog --stdout --title "$PN" --default-item "$DEFAULT_MIRROR" \
|
shellcheck grml-debootstrap:
grml-debootstrap#L857
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:857:- 0 0 $MIRROR)" || bailout
grml-debootstrap:857:+ 0 0 "$MIRROR")" || bailout
|
shellcheck grml-debootstrap:
grml-debootstrap#L863
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:863:- 0 0 $ISO)" || bailout
grml-debootstrap:863:+ 0 0 "$ISO")" || bailout
|
shellcheck grml-debootstrap:
grml-debootstrap#L1811
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:1811:- cp $VERBOSE "${CONFFILES}"/chroot-script "${MNTPOINT}"/bin/chroot-script
grml-debootstrap:1811:+ cp "$VERBOSE" "${CONFFILES}"/chroot-script "${MNTPOINT}"/bin/chroot-script
|
shellcheck grml-debootstrap:
grml-debootstrap#L1816
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:1816:- cp $VERBOSE "${CONFFILES}/config" "${MNTPOINT}"/etc/debootstrap/
grml-debootstrap:1816:+ cp "$VERBOSE" "${CONFFILES}/config" "${MNTPOINT}"/etc/debootstrap/
|
shellcheck grml-debootstrap:
grml-debootstrap#L1835
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:1835:- cp $VERBOSE "${_opt_packages:-$CONFFILES/$PACKAGES_FILE}" \
grml-debootstrap:1835:+ cp "$VERBOSE" "${_opt_packages:-$CONFFILES/$PACKAGES_FILE}" \
|
shellcheck grml-debootstrap:
grml-debootstrap#L1842
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:1842:- cp $VERBOSE "${_opt_debconf}" "${MNTPOINT}"/etc/debootstrap/debconf-selections
grml-debootstrap:1842:+ cp "$VERBOSE" "${_opt_debconf}" "${MNTPOINT}"/etc/debootstrap/debconf-selections
|
shellcheck grml-debootstrap:
grml-debootstrap#L1848
[shellcheck (suggestion)] reported by reviewdog 🐶
Raw Output:
grml-debootstrap:1848:- cp -a $VERBOSE "${_opt_chroot_scripts}"/* "${MNTPOINT}"/etc/debootstrap/chroot-scripts/
grml-debootstrap:1848:+ cp -a "$VERBOSE" "${_opt_chroot_scripts}"/* "${MNTPOINT}"/etc/debootstrap/chroot-scripts/
|
shellcheck grml-debootstrap:
chroot-script#L263
[shellcheck] reported by reviewdog 🐶
Possible misspelling: PACKAGES may not be assigned. Did you mean KPACKAGE? [SC2153](https://github.com/koalaman/shellcheck/wiki/SC2153)
Raw Output:
./chroot-script:263:9:info:Possible misspelling: PACKAGES may not be assigned. Did you mean KPACKAGE? [SC2153](https://github.com/koalaman/shellcheck/wiki/SC2153)
|
shellcheck grml-debootstrap:
grml-debootstrap#L276
[shellcheck] reported by reviewdog 🐶
Note that A && B || C is not if-then-else. C may run when A is true. [SC2015](https://github.com/koalaman/shellcheck/wiki/SC2015)
Raw Output:
./grml-debootstrap:276:37:info:Note that A && B || C is not if-then-else. C may run when A is true. [SC2015](https://github.com/koalaman/shellcheck/wiki/SC2015)
|
shellcheck grml-debootstrap:
grml-debootstrap#L281
[shellcheck] reported by reviewdog 🐶
Note that A && B || C is not if-then-else. C may run when A is true. [SC2015](https://github.com/koalaman/shellcheck/wiki/SC2015)
Raw Output:
./grml-debootstrap:281:39:info:Note that A && B || C is not if-then-else. C may run when A is true. [SC2015](https://github.com/koalaman/shellcheck/wiki/SC2015)
|
shellcheck grml-debootstrap:
grml-debootstrap#L286
[shellcheck] reported by reviewdog 🐶
Note that A && B || C is not if-then-else. C may run when A is true. [SC2015](https://github.com/koalaman/shellcheck/wiki/SC2015)
Raw Output:
./grml-debootstrap:286:36:info:Note that A && B || C is not if-then-else. C may run when A is true. [SC2015](https://github.com/koalaman/shellcheck/wiki/SC2015)
|
shellcheck grml-debootstrap:
grml-debootstrap#L795
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
Raw Output:
./grml-debootstrap:795:61:info:Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
|
shellcheck grml-debootstrap:
grml-debootstrap#L847
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
Raw Output:
./grml-debootstrap:847:64:info:Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
|
shellcheck grml-debootstrap:
grml-debootstrap#L857
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
Raw Output:
./grml-debootstrap:857:20:info:Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
|
shellcheck grml-debootstrap:
grml-debootstrap#L863
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
Raw Output:
./grml-debootstrap:863:20:info:Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
|
shellcheck grml-debootstrap:
grml-debootstrap#L1811
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
Raw Output:
./grml-debootstrap:1811:6:info:Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
|
shellcheck grml-debootstrap:
grml-debootstrap#L1816
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
Raw Output:
./grml-debootstrap:1816:6:info:Double quote to prevent globbing and word splitting. [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086)
|