Release 1.15.1

.github/workflows/ci.yml

@@ -18,6 +18,9 @@ defaults:
     shell: bash
 
 jobs:
+  tidy:
+    uses: taiki-e/workflows/.github/workflows/tidy-no-rust.yml@main
+
   test:
     strategy:
       fail-fast: false
@@ -28,7 +31,7 @@ jobs:
           # Note: Specifying the version of valgrind is not supported.
           # cargo-watch is supported by cargo-binstall (through quickinstall)
           - cargo-hack,cargo-llvm-cov,cargo-minimal-versions,parse-changelog,cargo-udeps,cargo-valgrind,cargo-deny,cross,nextest,protoc,shellcheck,shfmt,valgrind,wasm-pack,wasmtime,mdbook,mdbook-linkcheck,cargo-watch
-          - cargo-hack@0.5.23,cargo-llvm-cov@0.5.2,cargo-minimal-versions@0.1.8,parse-changelog@0.5.2,cargo-udeps@0.1.35,cargo-valgrind@2.1.0,cargo-deny@0.13.5,cross@0.2.4,nextest@0.9.11,protoc@3.21.9,shellcheck@0.8.0,shfmt@3.5.1,wasm-pack@0.10.3,wasmtime@3.0.0,mdbook@0.4.22,mdbook-linkcheck@0.7.6,cargo-watch@8.1.1
+          - cargo-hack@0.5.24,cargo-llvm-cov@0.5.2,cargo-minimal-versions@0.1.8,parse-changelog@0.5.2,cargo-udeps@0.1.35,cargo-valgrind@2.1.0,cargo-deny@0.13.5,cross@0.2.4,nextest@0.9.11,protoc@3.21.10,shellcheck@0.8.0,shfmt@3.5.1,wasm-pack@0.10.3,wasmtime@3.0.1,mdbook@0.4.22,mdbook-linkcheck@0.7.6,cargo-watch@8.1.1
           # Nextest supports basic version ranges as well
           - nextest@0.9
         include:
@@ -44,21 +47,3 @@ jobs:
       - uses: ./
         with:
           tool: ${{ matrix.tool }}
-
-  tidy:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          persist-credentials: false
-      - uses: taiki-e/install-action@shellcheck
-      - uses: taiki-e/install-action@shfmt
-      - run: npm install prettier
-      - run: shfmt -d $(git ls-files '*.sh')
-        if: always()
-      - run: npx prettier -c $(git ls-files '*.yml')
-        if: always()
-      - run: npx prettier -c $(git ls-files '*.js')
-        if: always()
-      - run: shellcheck $(git ls-files '*.sh')
-        if: always()

CHANGELOG.md

@@ -10,6 +10,12 @@ Note: In this file, do not use the hard wrap in the middle of a sentence for com
 
 ## [Unreleased]
 
+## [1.15.1] - 2022-12-03
+
+- Update `wasmtime@latest` to 3.0.1.
+- Update `protoc@latest` to 3.21.10.
+- Update `cargo-hack@latest` to 0.5.24.
+
 ## [1.15.0] - 2022-11-29
 
 - Support `cargo-deny`.
@@ -404,7 +410,8 @@ Note: In this file, do not use the hard wrap in the middle of a sentence for com
 
 Initial release
 
-[Unreleased]: https://github.com/taiki-e/install-action/compare/v1.15.0...HEAD
+[Unreleased]: https://github.com/taiki-e/install-action/compare/v1.15.1...HEAD
+[1.15.1]: https://github.com/taiki-e/install-action/compare/v1.15.0...v1.15.1
 [1.15.0]: https://github.com/taiki-e/install-action/compare/v1.14.7...v1.15.0
 [1.14.7]: https://github.com/taiki-e/install-action/compare/v1.14.6...v1.14.7
 [1.14.6]: https://github.com/taiki-e/install-action/compare/v1.14.5...v1.14.6

README.md

@@ -39,7 +39,7 @@ To install a specific version, use `@version` syntax:
 ```yaml
 - uses: taiki-e/install-action@v1
   with:
-    tool: cargo-hack@0.5.23
+    tool: cargo-hack@0.5.24
 ```
 
 To install multiple tools:

main.sh

@@ -177,7 +177,7 @@ for tool in "${tools[@]}"; do
         cargo-hack | cargo-llvm-cov | cargo-minimal-versions | parse-changelog)
             case "${tool}" in
                 # https://github.com/taiki-e/cargo-hack/releases
-                cargo-hack) latest_version="0.5.23" ;;
+                cargo-hack) latest_version="0.5.24" ;;
                 # https://github.com/taiki-e/cargo-llvm-cov/releases
                 cargo-llvm-cov) latest_version="0.5.2" ;;
                 # https://github.com/taiki-e/cargo-minimal-versions/releases
@@ -306,7 +306,7 @@ for tool in "${tools[@]}"; do
             ;;
         protoc)
             # https://github.com/protocolbuffers/protobuf/releases
-            latest_version="3.21.9"
+            latest_version="3.21.10"
             repo="protocolbuffers/protobuf"
             case "${version}" in
                 latest) version="${latest_version}" ;;
@@ -436,7 +436,7 @@ for tool in "${tools[@]}"; do
             ;;
         wasmtime)
             # https://github.com/bytecodealliance/wasmtime/releases
-            latest_version="3.0.0"
+            latest_version="3.0.1"
             repo="bytecodealliance/${tool}"
             case "${version}" in
                 latest) version="${latest_version}" ;;

tools/publish.sh

@@ -8,8 +8,8 @@ cd "$(dirname "$0")"/..
 # USAGE:
 #    ./tools/publish.sh <VERSION>
 #
-# Note:
-# - This script requires parse-changelog <https://github.com/taiki-e/parse-changelog>
+# Note: This script requires the following tools:
+# - parse-changelog <https://github.com/taiki-e/parse-changelog>
 
 bail() {
     echo >&2 "error: $*"

tools/tidy.sh

@@ -9,8 +9,14 @@ cd "$(dirname "$0")"/..
 #
 # Note: This script requires the following tools:
 # - shfmt
-# - prettier
 # - shellcheck
+# - npm (if any of YAML/JavaScript/JSON exists)
+# - jq and yq (if this repository uses bors)
+# - clang-format (if any of C/C++ exists)
+#
+# This script is shared with other repositories, so there may also be
+# checks for files not included in this repository, but they will be
+# skipped if the corresponding files do not exist.
 
 x() {
     local cmd="$1"
@@ -24,14 +30,33 @@ x() {
         "${cmd}" "$@"
     fi
 }
+check_diff() {
+    if [[ -n "${CI:-}" ]]; then
+        if ! git --no-pager diff --exit-code "$@"; then
+            should_fail=1
+        fi
+    else
+        if ! git --no-pager diff --exit-code "$@" &>/dev/null; then
+            should_fail=1
+        fi
+    fi
+}
 warn() {
-    echo >&2 "warning: $*"
+    if [[ -n "${GITHUB_ACTIONS:-}" ]]; then
+        echo "::warning::$*"
+    else
+        echo >&2 "warning: $*"
+    fi
+    should_fail=1
 }
 
 if [[ "${1:-}" == "-v" ]]; then
     shift
     verbose=1
 fi
+if [[ -n "${CI:-}" ]]; then
+    verbose=1
+fi
 if [[ $# -gt 0 ]]; then
     cat <<EOF
 USAGE:
@@ -40,24 +65,74 @@ EOF
     exit 1
 fi
 
-prettier=prettier
-if type -P npm &>/dev/null && type -P "$(npm bin)/prettier" &>/dev/null; then
-    prettier="$(npm bin)/prettier"
+# C/C++ (if exists)
+if [[ -n "$(git ls-files '*.c')$(git ls-files '*.cpp')" ]]; then
+    if [[ ! -e .clang-format ]]; then
+        warn "could not fount .clang-format in the repository root"
+    fi
+    if type -P clang-format &>/dev/null; then
+        x clang-format -i $(git ls-files '*.c') $(git ls-files '*.cpp')
+        check_diff $(git ls-files '*.c') $(git ls-files '*.cpp')
+    else
+        warn "'clang-format' is not installed"
+    fi
 fi
 
+# YAML/JavaScript/JSON (if exists)
+if [[ -n "$(git ls-files '*.yml')$(git ls-files '*.js')$(git ls-files '*.json')" ]]; then
+    if type -P npm &>/dev/null; then
+        if [[ ! -e node_modules/.bin/prettier ]]; then
+            x npm install prettier &>/dev/null
+        fi
+        x npx prettier -l -w $(git ls-files '*.yml') $(git ls-files '*.js') $(git ls-files '*.json')
+        check_diff $(git ls-files '*.yml') $(git ls-files '*.js') $(git ls-files '*.json')
+    else
+        warn "'npm' is not installed"
+    fi
+    if [[ -e .github/workflows/ci.yml ]] && grep -q '# tidy:needs' .github/workflows/ci.yml; then
+        if type -P jq &>/dev/null && type -P yq &>/dev/null; then
+            # shellcheck disable=SC2207
+            jobs_actual=($(yq '.jobs' .github/workflows/ci.yml | jq -r 'keys_unsorted[]'))
+            unset 'jobs_actual[${#jobs_actual[@]}-1]'
+            # shellcheck disable=SC2207
+            jobs_expected=($(yq -r '.jobs."ci-success".needs[]' .github/workflows/ci.yml))
+            if [[ "${jobs_actual[*]}" != "${jobs_expected[*]+"${jobs_expected[*]}"}" ]]; then
+                printf -v jobs '%s, ' "${jobs_actual[@]}"
+                sed -i "s/needs: \[.*\] # tidy:needs/needs: [${jobs%, }] # tidy:needs/" .github/workflows/ci.yml
+                check_diff .github/workflows/ci.yml
+                warn "please update 'needs' section in 'ci-success' job"
+            fi
+        else
+            warn "'jq' or 'yq' is not installed"
+        fi
+    fi
+fi
+if [[ -n "$(git ls-files '*.yaml')" ]]; then
+    warn "please use '.yml' instead of '.yaml' for consistency"
+    git ls-files '*.yaml'
+fi
+
+# Shell scripts
 if type -P shfmt &>/dev/null; then
     x shfmt -l -w $(git ls-files '*.sh')
+    check_diff $(git ls-files '*.sh')
 else
     warn "'shfmt' is not installed"
 fi
-if type -P "${prettier}" &>/dev/null; then
-    x "${prettier}" -l -w $(git ls-files '*.yml')
-    x "${prettier}" -l -w $(git ls-files '*.js')
-else
-    warn "'prettier' is not installed"
-fi
 if type -P shellcheck &>/dev/null; then
-    x shellcheck $(git ls-files '*.sh')
+    if ! x shellcheck $(git ls-files '*.sh'); then
+        should_fail=1
+    fi
+    if [[ -n "$(git ls-files '*Dockerfile')" ]]; then
+        # SC2154 doesn't seem to work on dockerfile.
+        if ! x shellcheck -e SC2148,SC2154,SC2250 $(git ls-files '*Dockerfile'); then
+            should_fail=1
+        fi
+    fi
 else
     warn "'shellcheck' is not installed"
 fi
+
+if [[ -n "${should_fail:-}" ]]; then
+    exit 1
+fi