Skip to content

feat(monorepo): Configure pnpm workspaces #5458

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.

Sign up for GitHub

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

Jump to bottom
Draft
DafyddLlyr wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

feat(monorepo): Configure pnpm workspaces #5458

DafyddLlyr wants to merge 12 commits into from

Conversation

@DafyddLlyr
Copy link
Contributor

@DafyddLlyr DafyddLlyr commented Oct 16, 2025 •
edited
Loading

  • feat: Setup pnpm workspace
  • fix: Update dockerfiles to build from root context for access to node_modules

To do...

  • Move overrides to root package.json
  • Pull request - install packages once and cache
  • Handle onlyBuiltDependencies warning - move to root

@DafyddLlyr DafyddLlyr changed the title dp/pnpm workspaces feat(monorepo): Configure pnpm workspaces Oct 16, 2025
@DafyddLlyr DafyddLlyr force-pushed the dp/pnpm-workspaces branch 2 times, most recently from 14186c5 to 2d64d7a Compare October 16, 2025 15:53
@DafyddLlyr DafyddLlyr force-pushed the dp/pnpm-workspaces branch 2 times, most recently from b952a71 to 2160526 Compare October 16, 2025 17:42
@DafyddLlyr DafyddLlyr force-pushed the dp/pnpm-workspaces branch 10 times, most recently from c2e53e8 to 5f876a3 Compare October 16, 2025 20:30
@DafyddLlyr
feat(ci): Handle e2e via the use of the --prefix flag
29be9c8 
- It probably makes more sense for this to be a package/e2e in future?
@DafyddLlyr
fix(ci): Use --prefix for Hasura tests also
07b3b01
@DafyddLlyr DafyddLlyr force-pushed the dp/pnpm-workspaces branch 5 times, most recently from 2686a00 to a80a195 Compare October 16, 2025 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DafyddLlyr