Skip to content

chore: add a base directory for TurboSnap #9258

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

Merged
merged 4 commits into from
Aug 23, 2023

Conversation

ethriel3695
Copy link
Contributor

@ethriel3695 ethriel3695 commented Aug 22, 2023

We need the storybookBaseDir to tell TurboSnap that the .storybook directory and commands are in a sub-directory.
workingDir only works for defining the current working directory.

I also did not realize that this repository was using the Vite builder.
Vite requires a plugin to work with TurboSnap.

It was created by one of the Core maintainers of Storybook, Ian.

Here is an example project of it working.
https://github.com/chromaui/vite-project

@cdr-bot cdr-bot bot added the community Pull Requests and issues created by the community. label Aug 22, 2023
@ethriel3695 ethriel3695 changed the title Add base dir for TurboSnap chore: Add base dir for TurboSnap Aug 22, 2023
@ethriel3695 ethriel3695 changed the title chore: Add base dir for TurboSnap chore: added a base directory for TurboSnap Aug 22, 2023
@ethriel3695 ethriel3695 changed the title chore: added a base directory for TurboSnap chore: add a base directory for TurboSnap Aug 22, 2023
@ethriel3695
Copy link
Contributor Author

@BrunoQuaresma
Can we talk about this one when you have a chance?

I did not realize the project was using Vite and TurboSnap won't work without the vite-plugin-turbosnap

@matifali matifali requested a review from BrunoQuaresma August 23, 2023 09:10
@ethriel3695
Copy link
Contributor Author

@BrunoQuaresma
It looks like Chromatic TurboSnap is finally picking up the preview-stats.json!

@BrunoQuaresma BrunoQuaresma merged commit 868ae06 into coder:main Aug 23, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Aug 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
community Pull Requests and issues created by the community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants