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

docs: Add info about packages not having the same command #1014

Merged
merged 4 commits into from
Apr 13, 2022

Conversation

eric-burel
Copy link
Contributor

Hi,

This looks like a trivial update but coming from Lerna, it's not obvious what will happen if some packages don't have a build step.

For instance my monorepo mixes NPM packages and full-application starters + docusaurus (https://github.com/VulcanJS/vulcan-npm).

The available scripts might vary accross package, some might not have "test" or even a "build" step.

As far as I can tell, lerna exec won't be that smart for instance and will try to run the same command for every package, failing if its not available to some, so it's worth being more explicit in the doc.

@vercel
Copy link

vercel bot commented Apr 7, 2022

@eric-burel is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

Co-authored-by: LeSuisse <thomas@gerbet.me>
@jaredpalmer jaredpalmer added the pr: automerge Kodiak will merge these automatically after checks pass label Apr 12, 2022
@jaredpalmer jaredpalmer changed the title Doc: Add info about packages not having the same command docs: Add info about packages not having the same command Apr 12, 2022
@vercel
Copy link

vercel bot commented Apr 12, 2022

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

Name Deployment Preview Updated
turbo-site Deployed View Apr 12, 2022 at 5:27PM (UTC)

@jaredpalmer jaredpalmer merged commit 368cd5e into vercel:main Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: automerge Kodiak will merge these automatically after checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants