From 5bc1a8ddff9ee4984f2a0761249d727467ac56ae Mon Sep 17 00:00:00 2001 From: "W. Evan Sheehan" Date: Wed, 1 Nov 2023 13:28:43 -0600 Subject: [PATCH] Remove old tests from webserver workflow The webserver container used to build a frontend with node and needed to run tests on the JavaScript. Now all of that is part of the app workflows and this is just for creating the container that runs the nginx reverse proxy, so we don't need those commented-out jobs anymore. --- .github/workflows/webserver.yaml | 39 -------------------------------- 1 file changed, 39 deletions(-) diff --git a/.github/workflows/webserver.yaml b/.github/workflows/webserver.yaml index d49eb67d..fcb5394e 100644 --- a/.github/workflows/webserver.yaml +++ b/.github/workflows/webserver.yaml @@ -41,45 +41,6 @@ jobs: run: npm ci - name: Lint run: npm run lint:code - # test: - # runs-on: ubuntu-latest - # permissions: - # pull-requests: write - # steps: - # - uses: actions/checkout@v4 - # - uses: actions/setup-node@v4 - # with: - # node-version-file: ".nvmrc" - # cache: "npm" - # cache-dependency-path: "services/api/package-lock.json" - # - name: Install dependencies - # working-directory: services/api - # run: npm ci - # - name: Test - # working-directory: services/api - # run: | - # npx playwright install-deps - # npx playwright install - # npm test - # - name: Code Coverage Report - # uses: irongut/CodeCoverageSummary@v1.3.0 - # with: - # filename: services/api/src/coverage/cobertura-coverage.xml - # badge: true - # fail_below_min: true - # format: markdown - # hide_branch_rate: false - # hide_complexity: true - # indicators: true - # output: both - # thresholds: "0 80" - # - name: Add Coverage PR Comment - # uses: marocchino/sticky-pull-request-comment@v2 - # if: github.event_name == 'pull_request' - # with: - # header: UI Coverage - # recreate: true - # path: code-coverage-results.md build: runs-on: ubuntu-latest needs: [lint]