clean up code, add comment, fix tests #103
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: check_make_vtadmin_web_proto | |
on: [push, pull_request] | |
permissions: read-all | |
jobs: | |
build: | |
name: Check Make VTAdmin Web Proto | |
runs-on: ubuntu-22.04 | |
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 | |
- name: Check out code | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' | |
uses: actions/checkout@v3 | |
- name: Check for changes in relevant files | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' | |
uses: frouioui/paths-filter@main | |
id: changes | |
with: | |
token: '' | |
filters: | | |
proto_changes: | |
- 'bootstrap.sh' | |
- 'tools/**' | |
- 'build.env' | |
- 'go.sum' | |
- 'go.mod' | |
- 'Makefile' | |
- 'go/vt/proto/**' | |
- 'proto/*.proto' | |
- 'web/vtadmin/src/proto/**' | |
- '.github/workflows/check_make_vtadmin_web_proto.yml' | |
- name: Set up Go | |
uses: actions/setup-go@v4 | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' && steps.changes.outputs.proto_changes == 'true' | |
with: | |
go-version: 1.21.1 | |
- name: Setup Node | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' && steps.changes.outputs.proto_changes == 'true' | |
uses: actions/setup-node@v3 | |
with: | |
# node-version should match package.json | |
node-version: '18.16.0' | |
- name: Install npm dependencies | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' && steps.changes.outputs.proto_changes == 'true' | |
run: npm ci | |
working-directory: ./web/vtadmin | |
- name: check_make_vtadmin_web_proto | |
if: steps.skip-workflow.outputs.skip-workflow == 'false' && steps.changes.outputs.proto_changes == 'true' | |
run: | | |
tools/check_make_vtadmin_web_proto.sh |