fix(hub-discussions): call response.text instead of response.json whe… #3457
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: PR Tests | |
| # Only run on non-draft PRs, and when PRs are synched | |
| on: | |
| pull_request: | |
| types: [opened, reopened, ready_for_review, synchronize] | |
| branches: | |
| - master | |
| - next | |
| - alpha | |
| - beta | |
| # If the PR has other files, a push w/ just .md or /docs | |
| # files will still cause tests to run | |
| paths-ignore: | |
| - "**.md" | |
| - "docs/**" | |
| jobs: | |
| build: | |
| # Don't run on draft PRs | |
| if: github.event.pull_request.draft == false | |
| runs-on: ${{ matrix.os }} | |
| # PRs will run across a wide range of versions & os's | |
| # This takes ~10 minutes, but it ensures we don't get any | |
| # weird regressions | |
| strategy: | |
| matrix: | |
| os: [ubuntu-latest, windows-latest] | |
| node: [14, 16] | |
| steps: | |
| - uses: actions/checkout@v3 | |
| - uses: actions/setup-node@v3 | |
| with: | |
| node-version: ${{ matrix.node }} | |
| # Npm 7 is needed for workspaces, and it shipped w/ Node 16 | |
| # But we want to test on downlevel versions of node so we | |
| # install npm7 directly | |
| - name: Npm 7 | |
| run: npm i -g npm@7 --registry=https://registry.npmjs.org | |
| - name: Install Dependencies | |
| run: npm ci | |
| - name: Build | |
| run: npm run build | |
| - name: Test in Chrome | |
| run: npm run test:chrome:ci | |
| # Only run FF on Linux b/c Windows is already really slow | |
| - name: Test in Firefox | |
| run: | | |
| if [ "$RUNNER_OS" == "Linux" ]; then | |
| npm run test:firefox:ci | |
| fi | |
| shell: bash | |
| - name: Test in Node | |
| run: npm run test:node | |
| - uses: codecov/codecov-action@v3 | |
| with: | |
| directory: ./coverage |