Merge new updates into main branch #28
Annotations
3 errors and 10 warnings
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.com/orgs/community/discussions/26680#discussioncomment-3252835
|
config.sample.sh#L1
[shellcheck] reported by reviewdog 🐶
Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.
Raw Output:
./config.sample.sh:1:1: error: Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive. (ShellCheck.SC2148)
|
|
get-last-status.sh#L20
[shellcheck] reported by reviewdog 🐶
Don't use ls | grep. Use a glob or a for loop with a condition to allow non-alphanumeric filenames.
Raw Output:
./get-last-status.sh:20:19: warning: Don't use ls | grep. Use a glob or a for loop with a condition to allow non-alphanumeric filenames. (ShellCheck.SC2010)
|
get-last-status.sh#L21
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.
Raw Output:
./get-last-status.sh:21:31: info: Double quote to prevent globbing and word splitting. (ShellCheck.SC2086)
|
zfs-replicate.sh#L19
[shellcheck] reported by reviewdog 🐶
Declare and assign separately to avoid masking return values.
Raw Output:
./zfs-replicate.sh:19:23: warning: Declare and assign separately to avoid masking return values. (ShellCheck.SC2155)
|
zfs-replicate.sh#L22
[shellcheck] reported by reviewdog 🐶
Declare and assign separately to avoid masking return values.
Raw Output:
./zfs-replicate.sh:22:23: warning: Declare and assign separately to avoid masking return values. (ShellCheck.SC2155)
|
zfs-replicate.sh#L26
[shellcheck] reported by reviewdog 🐶
$/${} is unnecessary on arithmetic variables.
Raw Output:
./zfs-replicate.sh:26:14: info: $/${} is unnecessary on arithmetic variables. (ShellCheck.SC2004)
|
zfs-replicate.sh#L28
[shellcheck] reported by reviewdog 🐶
Instead of 'let expr', prefer (( expr )) .
Raw Output:
./zfs-replicate.sh:28:9: info: Instead of 'let expr', prefer (( expr )) . (ShellCheck.SC2219)
|
zfs-replicate.sh#L33
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.
Raw Output:
./zfs-replicate.sh:33:10: info: Double quote to prevent globbing and word splitting. (ShellCheck.SC2086)
|
zfs-replicate.sh#L33
[shellcheck] reported by reviewdog 🐶
Double quote to prevent globbing and word splitting.
Raw Output:
./zfs-replicate.sh:33:22: info: Double quote to prevent globbing and word splitting. (ShellCheck.SC2086)
|
zfs-replicate.sh#L39
[shellcheck] reported by reviewdog 🐶
$/${} is unnecessary on arithmetic variables.
Raw Output:
./zfs-replicate.sh:39:19: info: $/${} is unnecessary on arithmetic variables. (ShellCheck.SC2004)
|
zfs-replicate.sh#L41
[shellcheck] reported by reviewdog 🐶
Instead of 'let expr', prefer (( expr )) .
Raw Output:
./zfs-replicate.sh:41:13: info: Instead of 'let expr', prefer (( expr )) . (ShellCheck.SC2219)
|
This job failed
Loading