layer5io / layer5io/sistent

[workspace] Review hoisting dependencies in yarn/lerna workspace

Open
#126 1 comment 0 reactions 1 assignee View on GitHub

@nebula-aac is already working on this.

Since Oct 6, 2023.

kind/bug priority/high
Dominant language
TypeScript
Stars
137
Forks
239
Avg merge
4d 14h
Merged PRs (30d)
5

Description

UPDATE

The current setup is Yarn 3.6.3 which does not use the PnP, which means, we're still using node_modules. After several tries, there seems to be issues with multiple same packages in the same monorepo, and yarn doesn't seem to be respecting this behavior.

The alternative is create the isolated apps and treat them as such, and don't let them build with the monorepo.

Updating this issue to keep track.

Issue:

Before I made the changes, I was able to add the internal packages as a dependencies, but I ran into an issue when I packaged and published a package with workspace in the version.

The packages are now pinned to the latest version but it does not help in this development, because it is easier test via hoisting, rather than pinning a version and publishing a version to test the changes.

Opening a ticket to track this behavior.


Contributor Guides and Resources

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.