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

Switch to monorepo #321

Merged
merged 1 commit into from
Jan 20, 2024
Merged

Switch to monorepo #321

merged 1 commit into from
Jan 20, 2024

Conversation

kitsuyui
Copy link
Owner

  • Add pnpm-workspace.yaml
  • Move current project items into packages/simple/
  • Install turborepo

@kitsuyui kitsuyui force-pushed the monorepo branch 5 times, most recently from 414ae29 to 24cbd25 Compare January 20, 2024 14:17
- Add pnpm-workspace.yaml
- Move current project items into packages/simple/
- Install turborepo
@kitsuyui kitsuyui merged commit 5f254bc into main Jan 20, 2024
5 checks passed
@kitsuyui kitsuyui deleted the monorepo branch January 20, 2024 14:24
kitsuyui added a commit that referenced this pull request Feb 2, 2024
I forgot to add the workspaces field to package.json in #321 .
So Renovate didn't pick up the change in monorepo configurations properly and causes the build to fail.

```
ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up to date with packages/simple/package.json
```

This commit adds the workspaces field to package.json and updates pnpm-lock.yaml accordingly.
kitsuyui added a commit that referenced this pull request Jan 4, 2025
When I migrated to monorepo in #321, I should have removed the lock files that were no longer in use.
It is sufficient to have the lock file in the package root of the monorepo, so the lock files other than that should be removed.
@kitsuyui kitsuyui mentioned this pull request Jan 4, 2025
kitsuyui added a commit that referenced this pull request Jan 4, 2025
When I migrated to monorepo in #321, I should have removed the lock files that were no longer in use.
It is sufficient to have the lock file in the package root of the monorepo, so the lock files other than that should be removed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant