Skip to content

Topic/shellcheck#61

Closed
danielocfb wants to merge 6 commits intolibbpf:masterfrom
danielocfb:topic/shellcheck
Closed

Topic/shellcheck#61
danielocfb wants to merge 6 commits intolibbpf:masterfrom
danielocfb:topic/shellcheck

Conversation

@danielocfb
Copy link
Collaborator

test

teknoraver and others added 6 commits October 26, 2022 15:45
Signed-off-by: Matteo Croce <teknoraver@meta.com>
Signed-off-by: Matteo Croce <teknoraver@meta.com>
Signed-off-by: Matteo Croce <teknoraver@meta.com>
This adds a GitHub action which runs a shell linter on all the .sh files.
ludeeus/action-shellcheck is just an action which relies on
https://github.com/koalaman/shellcheck which is a well known linter.

Disable style suggestions SC1091, SC2181 and SC2001 because they are not
worth fixing in this repository.

    ~/src/libbpf-ci$ find * -type f -name '*.sh' |xargs shellcheck -e SC1091,SC2181,SC2001
    ~/src/libbpf-ci$ echo $?
    0

Signed-off-by: Matteo Croce <teknoraver@meta.com>
@danielocfb danielocfb closed this Oct 26, 2022
teknoraver added a commit to teknoraver/libbpf that referenced this pull request Oct 27, 2022
Run shellckeck linter in a github action,
as in libbpf/ci#61

Signed-off-by: Matteo Croce <teknoraver@meta.com>
teknoraver added a commit to teknoraver/libbpf that referenced this pull request Oct 27, 2022
Run shellckeck linter in a github action,
as in libbpf/ci#61

Signed-off-by: Matteo Croce <teknoraver@meta.com>
teknoraver added a commit to teknoraver/libbpf that referenced this pull request Oct 27, 2022
Run shellckeck linter in a github action,
as in libbpf/ci#61

Signed-off-by: Matteo Croce <teknoraver@meta.com>
anakryiko pushed a commit to libbpf/libbpf that referenced this pull request Oct 27, 2022
Run shellckeck linter in a github action,
as in libbpf/ci#61

Signed-off-by: Matteo Croce <teknoraver@meta.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants