wmantly
ea75e94b3e
Renaming the job broke the master protection rule, which requires a check named exactly 'Syntax check bootstrap.js'. The job still checks both bootstrap scripts, just under the protected name.
51 lines
1.3 KiB
YAML
51 lines
1.3 KiB
YAML
name: Lint
|
|
|
|
# theta-suite has no app code of its own to unit-test (it orchestrates the
|
|
# proxy/sso-manager-node submodules) -- this checks the one thing that can
|
|
# actually break silently: setup.sh and bootstrap.js, plus a static
|
|
# consistency check on the config bootstrap.js generates for jump-host
|
|
# (test/check_jump_ldap_tls.js).
|
|
on:
|
|
pull_request:
|
|
branches:
|
|
- master
|
|
push:
|
|
branches-ignore:
|
|
- master
|
|
|
|
jobs:
|
|
shellcheck:
|
|
name: Shellcheck setup.sh
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- name: Checkout code
|
|
uses: actions/checkout@v4
|
|
|
|
- name: Syntax check
|
|
run: bash -n setup.sh
|
|
|
|
- name: Shellcheck
|
|
run: shellcheck -S warning setup.sh
|
|
|
|
bootstrap-syntax:
|
|
# Keep this job name stable: branch protection on master requires a status
|
|
# check named exactly "Syntax check bootstrap.js".
|
|
name: Syntax check bootstrap.js
|
|
runs-on: ubuntu-latest
|
|
steps:
|
|
- name: Checkout code
|
|
uses: actions/checkout@v4
|
|
|
|
- name: Setup Node.js
|
|
uses: actions/setup-node@v4
|
|
with:
|
|
node-version: 22.x
|
|
|
|
- name: Syntax check
|
|
run: |
|
|
node --check bootstrap/bootstrap.js
|
|
node --check bootstrap/site-join.js
|
|
|
|
- name: Jump-host LDAP config consistency
|
|
run: node test/check_jump_ldap_tls.js
|