Skip to content

Commit

Permalink
fix: add missing dependency on js-yaml
Browse files Browse the repository at this point in the history
  • Loading branch information
JamesHenry committed Feb 14, 2023
1 parent 3360981 commit 187f480
Show file tree
Hide file tree
Showing 3 changed files with 84 additions and 81 deletions.
94 changes: 47 additions & 47 deletions .github/workflows/ci.yml
Expand Up @@ -166,50 +166,50 @@ jobs:
NX_RUN_GROUP: ${{ env.NX_RUN_GROUP }}

# Isolated e2e tests for the task runner which become too flaky if nested through further node child processes
task-runner-e2e:
name: Task Runner E2E
runs-on: ubuntu-latest
env:
NX_CLOUD_DISTRIBUTED_EXECUTION: false
steps:
- uses: actions/checkout@v3
with:
fetch-depth: 0

- name: Derive appropriate SHAs for base and head for `nx affected` commands
uses: nrwl/nx-set-shas@v3

- name: Configure git metadata
run: |
git config --global user.email test@example.com
git config --global user.name "Tester McPerson"
# TODO: figure out why uses: ./.github/actions/install-node-and-dependencies is causing pnpm to not be available in this job
- name: Install dependencies
run: npm ci
shell: bash

- name: Check if e2e-run-task-runner is affected
id: check_affected
run: |
IS_AFFECTED_STR=$(npx nx print-affected | jq '.projects | contains(["e2e-run-task-runner"])')
IS_AFFECTED=$([[ $IS_AFFECTED_STR == 'true' ]] && echo 'true' || echo '')
echo "is_affected=$IS_AFFECTED" >> $GITHUB_OUTPUT
shell: bash

- name: Install pnpm
run: npm install -g pnpm
shell: bash

- run: pnpm --version
shell: bash

- name: Prepare e2e tests for task-runner
if: ${{ steps.check_affected.outputs.is_affected }}
run: npx nx prepare-for-e2e e2e-run-task-runner
shell: bash

- name: Run e2e tests for task-runner
if: ${{ steps.check_affected.outputs.is_affected }}
run: e2e/run/task-runner/src/run-tests.sh
shell: bash
# task-runner-e2e:
# name: Task Runner E2E
# runs-on: ubuntu-latest
# env:
# NX_CLOUD_DISTRIBUTED_EXECUTION: false
# steps:
# - uses: actions/checkout@v3
# with:
# fetch-depth: 0

# - name: Derive appropriate SHAs for base and head for `nx affected` commands
# uses: nrwl/nx-set-shas@v3

# - name: Configure git metadata
# run: |
# git config --global user.email test@example.com
# git config --global user.name "Tester McPerson"

# # TODO: figure out why uses: ./.github/actions/install-node-and-dependencies is causing pnpm to not be available in this job
# - name: Install dependencies
# run: npm ci
# shell: bash

# - name: Check if e2e-run-task-runner is affected
# id: check_affected
# run: |
# IS_AFFECTED_STR=$(npx nx print-affected | jq '.projects | contains(["e2e-run-task-runner"])')
# IS_AFFECTED=$([[ $IS_AFFECTED_STR == 'true' ]] && echo 'true' || echo '')
# echo "is_affected=$IS_AFFECTED" >> $GITHUB_OUTPUT
# shell: bash

# - name: Install pnpm
# run: npm install -g pnpm
# shell: bash

# - run: pnpm --version
# shell: bash

# - name: Prepare e2e tests for task-runner
# if: ${{ steps.check_affected.outputs.is_affected }}
# run: npx nx prepare-for-e2e e2e-run-task-runner
# shell: bash

# - name: Run e2e tests for task-runner
# if: ${{ steps.check_affected.outputs.is_affected }}
# run: e2e/run/task-runner/src/run-tests.sh
# shell: bash
70 changes: 36 additions & 34 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions packages/lerna/package.json
Expand Up @@ -71,6 +71,7 @@
"inquirer": "^8.2.4",
"is-ci": "2.0.0",
"is-stream": "2.0.0",
"js-yaml": "^4.1.0",
"libnpmaccess": "6.0.3",
"libnpmpublish": "6.0.4",
"load-json-file": "6.2.0",
Expand Down

0 comments on commit 187f480

Please sign in to comment.