Skip to content

Remove workspaces#34

Merged
ericclemmons merged 20 commits into
masterfrom
34-remove-workspaces
Mar 10, 2019
Merged

Remove workspaces#34
ericclemmons merged 20 commits into
masterfrom
34-remove-workspaces

Conversation

@ericclemmons
Copy link
Copy Markdown
Owner

@ericclemmons ericclemmons commented Mar 10, 2019

Before #28 or #21, we can fix #14 by:

  1. Removing yarn workspaces.
  2. Keep yarn link for polydev.
  3. Move each package into examples with a package.json.
  4. Move packages/polydev to src.

Running an example should require cd into that directory, yarn and polydev.

By doing this, we can ensure that hoisted modules don't affect development/examples.

Also, np will behave as expected.

@ericclemmons ericclemmons merged commit d47b538 into master Mar 10, 2019
@ericclemmons ericclemmons mentioned this pull request Mar 11, 2019
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.

yarn release doesn't work

1 participant