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

Add steps to link for local testing #347

Conversation

codykaup
Copy link
Contributor

@codykaup codykaup commented Nov 1, 2024

It took me a while to figure out how to setup a local version of the VTA with a local version of the CLI for testing. This adds the steps I used to save others some time!

@codykaup codykaup added skip-release Auto: Preserve the current version when merged patch Auto: Increment the patch version when merged labels Nov 1, 2024
@codykaup codykaup self-assigned this Nov 1, 2024
@codykaup codykaup added the documentation Classification: Changes only affect the documentation label Nov 1, 2024
Copy link
Member

@ghengeveld ghengeveld left a comment

Choose a reason for hiding this comment

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

I didn't know you could specify a path. Usually you'd run yarn link in the source repo (VTA) and then yarn link @chromatic-com/storybook in the target repo. Did you try that?

@codykaup
Copy link
Contributor Author

I didn't know you could specify a path. Usually you'd run yarn link in the source repo (VTA) and then yarn link @chromatic-com/storybook in the target repo. Did you try that?

I did but they changed it in Yarn 4. Should I add steps for both?

@codykaup
Copy link
Contributor Author

I didn't know you could specify a path. Usually you'd run yarn link in the source repo (VTA) and then yarn link @chromatic-com/storybook in the target repo. Did you try that?

I did but they changed it in Yarn 4. Should I add steps for both?

Going to merge this in and we can add those steps later if it becomes a problem!

@codykaup codykaup merged commit ec0a952 into main Nov 19, 2024
10 of 11 checks passed
@codykaup codykaup deleted the cody/cap-2346-write-up-doc-on-how-to-setup-vta-locally-for-testing-a-new branch November 19, 2024 15:46
@ghengeveld
Copy link
Member

🚀 PR was released in v3.2.3 🚀

@ghengeveld ghengeveld added the released Verdict: This issue/pull request has been released label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Classification: Changes only affect the documentation patch Auto: Increment the patch version when merged released Verdict: This issue/pull request has been released skip-release Auto: Preserve the current version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants