Skip to content

Commit

Permalink
get-port override not needed since cypress@v11 (#245)
Browse files Browse the repository at this point in the history
Looks like since about cypress 11, node modules are being preloaded into
a custom v8 snapshot which makes our override a noop.
  • Loading branch information
mhan83 committed May 4, 2023
1 parent f848b78 commit a49b030
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 426 deletions.
10 changes: 0 additions & 10 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -110,11 +110,6 @@ jobs:
run: |
Write-Output "cypress_version=$((Get-Content ./package.json -Raw | ConvertFrom-Json).dependencies.cypress)" >> $env:GITHUB_ENV
- name: Override Dependencies
run: |
rm -r bundle/Cache/${{ env.cypress_version }}/Cypress/resources/app/node_modules/get-port
cp -r vendor/get-port bundle/cache/${{ env.cypress_version }}/Cypress/resources/app/node_modules
- name: Archive Bundle
uses: azure/powershell@v1
with:
Expand Down Expand Up @@ -168,11 +163,6 @@ jobs:
CYPRESS_VERSION=$(< package-lock.json jq -r '.dependencies["cypress"].version')
echo "cypress_version=$CYPRESS_VERSION" >> $GITHUB_ENV
- name: Override Dependencies
run: |
rm -r bundle/Cache/${{ env.cypress_version }}/Cypress.app/Contents/Resources/app/node_modules/get-port
cp -r vendor/get-port bundle/Cache/${{ env.cypress_version }}/Cypress.app/Contents/Resources/app/node_modules/
- name: Archive Bundle
run: zip --symlinks -r cypress-macos-amd64.zip bundle/

Expand Down
62 changes: 0 additions & 62 deletions vendor/get-port/index.d.ts

This file was deleted.

179 changes: 0 additions & 179 deletions vendor/get-port/index.js

This file was deleted.

9 changes: 0 additions & 9 deletions vendor/get-port/license

This file was deleted.

49 changes: 0 additions & 49 deletions vendor/get-port/package.json

This file was deleted.

0 comments on commit a49b030

Please sign in to comment.