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

Use studio-plugin version in package.json instead of wildcard #355

Merged
merged 2 commits into from
Aug 29, 2023

Conversation

alextaing
Copy link
Contributor

@alextaing alextaing commented Aug 29, 2023

In npm workspaces, you can use * to point to a local dependency (i.e. another package), which is how we add studio-plugin as a dependency of studio. This dependency is preserved as a symlink in the root package-lock.json. Since the dependency version of studio-plugin in the studio package.json was *, there was not a concrete version of studio-plugin specified, so users had to explicitly install studio and studio-plugin. This PR finds that we can still specify a version of studio-plugin in the studio package.json and still preserve the symlink for local development.

J-SLAP-2895
TEST=manual

@oshi97
Copy link
Contributor

oshi97 commented Aug 29, 2023

very pog, please make a corresponding item!

Copy link
Contributor

@oshi97 oshi97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!!

@alextaing alextaing merged commit 2244f4d into main Aug 29, 2023
13 checks passed
@alextaing alextaing deleted the dev/link-studio-and-plugin branch August 29, 2023 18:02
tmeyer2115 added a commit that referenced this pull request Aug 31, 2023
## Changes
- Studio's startup time is now considerably faster. This is due to an
improvement in the logic for parsing Component and Template files.
(#357)
- The Studio Plugin is now a direct, explicit dependency of Studio. That
means users only have to install `@yext/studio`. They don't need to
worry about manually installing `@yext/studio-plugin` as well. (#355)
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.

3 participants