[release-20.0] VTAdmin: Fix serve-handler's path-to-regexp dep and add default schema refresh (#16778) #3897
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
name: vtadmin-web linting + formatting | |
# In specifying the 'paths' property, we need to include the path to this workflow .yml file. | |
# See https://github.community/t/trigger-a-workflow-on-change-to-the-yml-file-itself/17792/4) | |
on: | |
push: | |
paths: | |
- '.github/workflows/vtadmin_web_lint.yml' | |
- 'web/vtadmin/**' | |
pull_request: | |
paths: | |
- '.github/workflows/vtadmin_web_lint.yml' | |
- 'web/vtadmin/**' | |
permissions: read-all | |
jobs: | |
lint: | |
runs-on: ubuntu-latest | |
steps: | |
- name: Skip CI | |
run: | | |
if [[ "${{contains( github.event.pull_request.labels.*.name, 'Skip CI')}}" == "true" ]]; then | |
echo "skipping CI due to the 'Skip CI' label" | |
exit 1 | |
fi | |
- name: Check if workflow needs to be skipped | |
id: skip-workflow | |
run: | | |
skip='false' | |
if [[ "${{github.event.pull_request}}" == "" ]] && [[ "${{github.ref}}" != "refs/heads/main" ]] && [[ ! "${{github.ref}}" =~ ^refs/heads/release-[0-9]+\.[0-9]$ ]] && [[ ! "${{github.ref}}" =~ "refs/tags/.*" ]]; then | |
skip='true' | |
fi | |
echo Skip ${skip} | |
echo "skip-workflow=${skip}" >> $GITHUB_OUTPUT | |
- uses: actions/checkout@v4 | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' | |
- uses: actions/setup-node@v4 | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' | |
with: | |
# node-version should match package.json | |
node-version: '20.12.2' | |
- name: Install dependencies | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' | |
run: cd ./web/vtadmin && npm ci | |
# Using "if: always()" means each step will run, even if a previous | |
# step fails. This is nice because, for example, we want stylelint and | |
# prettier to run even if eslint fails. | |
# | |
# An undesirable secondary effect of this is these steps | |
# will run even if the install, etc. steps fail, which is... weird. | |
# A nice enhancement is to parallelize these steps into jobs, with the | |
# trade-off of more complexity around sharing npm install artifacts. | |
- name: Run eslint | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' && always() | |
run: cd ./web/vtadmin && npm run lint:eslint | |
- name: Run stylelint | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' && always() | |
run: cd ./web/vtadmin && npm run lint:stylelint -- -f verbose | |
- name: Run prettier | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' && always() | |
run: cd ./web/vtadmin && npm run lint:prettier | |
# Cancel pending and in-progress runs of this workflow if a newer ref is pushed to CI. | |
concurrency: | |
group: ${{ github.workflow }}-${{ github.ref }} | |
cancel-in-progress: true |