From 86bc0f4b83390df16b1e9c0199dfd89e0a7ccfcb Mon Sep 17 00:00:00 2001 From: Masaya Suzuki <15100604+massongit@users.noreply.github.com> Date: Fri, 10 Jan 2025 23:29:26 +0900 Subject: [PATCH] fix fix Update reviewdog.yml Update entrypoint.sh Create broken_yaml.yaml test --- .github/workflows/broken_yaml.yaml | 6 ++++++ .github/workflows/reviewdog.yml | 11 +++++++++++ 2 files changed, 17 insertions(+) create mode 100644 .github/workflows/broken_yaml.yaml diff --git a/.github/workflows/broken_yaml.yaml b/.github/workflows/broken_yaml.yaml new file mode 100644 index 0000000..fb7f83d --- /dev/null +++ b/.github/workflows/broken_yaml.yaml @@ -0,0 +1,6 @@ +on: push +jobs: + linux: + runs-on: ubuntu-latest + steps: + - run: foo: diff --git a/.github/workflows/reviewdog.yml b/.github/workflows/reviewdog.yml index ca969fc..00d3ee8 100644 --- a/.github/workflows/reviewdog.yml +++ b/.github/workflows/reviewdog.yml @@ -17,6 +17,17 @@ jobs: uses: ./ with: tool_name: actionlint + fail_level: error + - run: | + cd dir; process *; cd ..; # Use subshells instead + [[ -z $(find /tmp | grep mpg) ]] # Use grep -q instead + a >> log; b >> log; c >> log # Use a redirection block instead + echo "The time is `date`" # Use $() instead + cd dir; process *; cd ..; # Use subshells instead + echo $[1+2] # Use standard $((..)) instead of old $[] + echo $(($RANDOM % 6)) # Don't use $ on variables in $((..)) + echo "$(date)" # Useless use of echo + cat file | grep foo # Useless use of cat reviewdog: name: runner / reviewdog