Skip to content

Commit 9925d18

Browse files
committed
ci(gh-actions): bash strict mode
1 parent ac66467 commit 9925d18

File tree

3 files changed

+15
-12
lines changed

3 files changed

+15
-12
lines changed

.github/workflows/automerge.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ jobs:
1515
- name: Enable auto-merge for Dependabot PRs
1616
shell: bash
1717
run: |
18+
set -euo pipefail
19+
IFS=$'\n\t'
1820
# Checking the PR title is a poor substitute for the actual PR changes
1921
# but as long as this is used only with dependabot PRs,
2022
# it should be safe to assume that the title is not misleading.

.github/workflows/jslint.yml

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,14 @@ jobs:
1919
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
2020
- name: Install JSLint
2121
shell: bash
22-
run: sudo npm install -g jslint
22+
run: |
23+
set -euo pipefail
24+
IFS=$'\n\t'
25+
sudo npm install -g jslint@0.12.1
26+
sudo npm outdated --global
2327
- name: Run JSLint
2428
shell: bash
25-
run: jslint --browser javascripts/*.js
29+
run: |
30+
set -euo pipefail
31+
IFS=$'\n\t'
32+
jslint --browser javascripts/*.js

.github/workflows/yamllint.yml

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,9 @@ jobs:
1717
steps:
1818
- name: Check out
1919
uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
20-
- name: Install YAMLLint
21-
shell: bash
22-
run: |
23-
pip install yamllint==1.37.1
24-
msg="$(pip list --outdated | grep -e yamllint || true)"
25-
if [ -n "${msg}" ]; then
26-
>&2 echo "ERROR: outdated: ${msg}"
27-
exit 1
28-
fi
2920
- name: Run YAMLLint
3021
shell: bash
31-
run: "yamllint -d '{extends: default, rules: {line-length: disable}}' ."
22+
run: |
23+
set -euo pipefail
24+
IFS=$'\n\t'
25+
yamllint -d '{extends: default, rules: {line-length: disable}}' .

0 commit comments

Comments
 (0)