Skip to content

fix(storybook): ensure 'storybook' dep is installed for non-crystal workspaces#29235

Merged
Coly010 merged 1 commit intomasterfrom
storybook/ensure-storybook-dep
Dec 6, 2024
Merged

fix(storybook): ensure 'storybook' dep is installed for non-crystal workspaces#29235
Coly010 merged 1 commit intomasterfrom
storybook/ensure-storybook-dep

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Dec 6, 2024

Current Behavior

The storybook package is required to be installed to correctly find paths correctly.
It acts almost like a shim.
It was not being installed for non-crystal workspaces

Expected Behavior

Ensure storybook is installed

@Coly010 Coly010 requested a review from a team as a code owner December 6, 2024 15:54
@Coly010 Coly010 requested a review from mandarini December 6, 2024 15:54
@Coly010 Coly010 self-assigned this Dec 6, 2024
@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Dec 6, 2024 3:57pm

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Dec 6, 2024

@Coly010 Coly010 merged commit 288193b into master Dec 6, 2024
@Coly010 Coly010 deleted the storybook/ensure-storybook-dep branch December 6, 2024 16:39
FrozenPandaz pushed a commit that referenced this pull request Dec 6, 2024
…orkspaces (#29235)

## Current Behavior
The `storybook` package is required to be installed to correctly find
paths correctly.
It acts almost like a shim.
It was not being installed for non-crystal workspaces

## Expected Behavior
Ensure `storybook` is installed

(cherry picked from commit 288193b)
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants