Skip to content

Commit

Permalink
Try adding Cypress back (#82)
Browse files Browse the repository at this point in the history
* Add Cypress back

* force run on pr

* Trigger change in pkg

* Add back step "Set up Cypress binary cache"
  • Loading branch information
enjikaka authored Mar 6, 2024
1 parent c81ef12 commit e054d26
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ on:
- packages/true-time/**
- packages/player/**
pull_request:
branches: [ "main"]
branches: [ "main" ]
paths:
- packages/true-time/**
- packages/player/**
Expand All @@ -26,7 +26,6 @@ jobs:
- name: Checkout
uses: actions/checkout@v4
- uses: ./.github/actions/setup
- uses: ./.github/actions/build-projects
- name: Prepare test user for player tests
id: json
run: |
Expand All @@ -38,6 +37,11 @@ jobs:
echo "encoded=\"$encoded_json\"" >> $GITHUB_OUTPUT
env:
TEST_USER: ${{ secrets.PLAYER_TEST_USER }}
- name: Set up Cypress binary cache
uses: actions/cache@v4
with:
path: ~/.cache/Cypress
key: ${{ runner.os }}-cypress-${{ hashFiles('pnpm-lock.yaml') }}
- name: Cypress run
uses: cypress-io/github-action@v6
env:
Expand Down
2 changes: 1 addition & 1 deletion packages/player/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ To load TIDAL Player into a project locally without publishing to npm; using `pn

1. Open the package.json in the project. (in the case of webclient, the root one)
2. Add or edit a `"resolutions"` property on the top level. (this is an object)
3. Add an entry for @tidal-music/player like so: `"@tidal-music/player": "file:/Users/yourusername/dev/tidal-sdk-web/packages/player"`
3. Add an entry for @tidal-music/player like so: `"@tidal-music/player": "file:/Users/<your-username>/dev/tidal-sdk-web/packages/player"`
4. Run `npm/yarn/pnpm install` in your destination project.

!! Remove the entry in `"resolution"` and run `npm/yarn/pnpm install` again when you are done testing to have the NPM version of the package load instead.
Expand Down

0 comments on commit e054d26

Please sign in to comment.