Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add workspace support for pnpm #1144

Open
MikeMcC399 opened this issue Mar 8, 2024 · 10 comments
Open

Add workspace support for pnpm #1144

MikeMcC399 opened this issue Mar 8, 2024 · 10 comments
Labels
type: feature feature request

Comments

@MikeMcC399
Copy link
Collaborator

What would you like

When the cypress-io/github-action processes a project with a pnpm-lock.yaml lockfile from the pnmp package manager it should be able to interpret pnpm workspaces correctly and run Cypress from a pnpm workspace, including dependency installation and caching.

For example, the following should work:

      - name: Cypress test
        uses: cypress-io/github-action@v6
        with:
          working-directory: examples/start-and-pnpm-workspaces/packages/workspace-1

This fails because Cypress can't find pnpm-lock.yaml to install dependencies.

Why is this needed?

The workspace of a pnpm project does not contain any pnpm-lock.yaml file and cypress-io/github-action does not read and interpret any pnpm-workspace.yaml file in the root of a pnpm project using workspaces. This means that the action needs to be applied twice, once in the root and then again in the workspace in order to run Cypress in a pnpm workspace. (See pnpm workspaces.) This is error prone.

The cypress-io/github-action already provides support for Yarn workspaces. Not providing this functionality for pnpm projects with workspaces is inconsistent.

The benefit for users of pnpm projects with workspaces is that the workflows using GitHub Actions can be written in a more simpler way. This reduces the risk of introducing errors especially with caching the Cypress binary.

Background

pnpm has built-in support for monorepositories (AKA multi-package repositories, multi-project repositories, or monolithic repositories).

pnpm repos using workspaces and Cypress are typically structured similar to the following E2E example:

# repo root
/node_modules
/packages
/package.json
/pnpm-lock.yaml
/pnpm-workspace.yaml

# workspace 1
/packages/ws1
/packages/ws1/cypress/e2e/spec.cy.js
/packages/ws1/node_modules
/packages/ws1/cypress.config.js
/packages/ws1/package.json

# workspace 2
etc.

Other


pnpm workspaces

If you are using pnpm workspaces you need to install dependencies and run Cypress tests in a workspace in separate steps. The snippet below shows this principle.

      ...
      - name: Install dependencies
        uses: cypress-io/github-action@v6
        with:
          working-directory: examples/start-and-pnpm-workspaces
          runTests: false

      - name: Cypress test
        uses: cypress-io/github-action@v6
        with:
          install: false
          working-directory: examples/start-and-pnpm-workspaces/packages/workspace-1
        ...

pnpm workspaces example

See the example project start-and-pnpm-workspaces and the example-start-and-pnpm-workspaces.yml workflow for a full working example including pnpm caching.

@osmestad

This comment was marked as resolved.

@MikeMcC399
Copy link
Collaborator Author

@osmestad

@osmestad

This comment was marked as resolved.

@MikeMcC399
Copy link
Collaborator Author

@osmestad

Good you found your problem!

I don't have privileges to delete other people's posts. You can however delete your own posts or select Hide from the three-dot menu and hide your post(s) from being displayed.

@misha-erm
Copy link

hello, I've just noticed one more problem with pnpm workspace. After latest release of action it shows that there is cypress '13.9.0' cached but later it fails to find binaries for '13.8.1' as our tests use it. I followed start-and-pnpm-workspaces and it worked like a charm until today )

Seems a bit similar to #971

image image

Thank you 🙇🏻

@MikeMcC399
Copy link
Collaborator Author

@misha-erm

Please open a new issue if you are asking for help. This issue is for the pnpm workspace support enhancement request, not any bug reports about the existing implementation or documentation.

@misha-erm
Copy link

misha-erm commented May 11, 2024

I added a comment there as I thought that might be in scope of "workspace support"
If you think that's not related, I can move it to separate issue, sure

if you are asking for help

For now I just upgraded local cypress version to match the version installed by the github action. Just was a bit surprised that action can't read correct version from pnpm-lock

@MikeMcC399
Copy link
Collaborator Author

@misha-erm

I tested start-and-pnpm-workspaces with Cypress 13.8.0 in workspace-1 and 13.9.0 in workspace-2 using the workflow example-start-and-pnpm-workspaces.yml and it worked correctly, installing and caching both versions.

If you need to have mixed Cypress versions in your pnpm workspaces and it is not working for you, then we can look at it in a separate issue. We would need to see your workflow and your lock file for that.

@misha-erm
Copy link

got it. I will keep an eye on it and if it reproduces again will create new issue with reproduction

@MikeMcC399
Copy link
Collaborator Author

MikeMcC399 commented May 11, 2024

@misha-erm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature feature request
Projects
None yet
Development

No branches or pull requests

3 participants