-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Yarn 4 #6215
base: main
Are you sure you want to change the base?
Yarn 4 #6215
Conversation
Build successful! π |
# Conflicts: # yarn.lock
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
1 similar comment
Build successful! π |
Build successful! π |
1 similar comment
Build successful! π |
@@ -0,0 +1,106 @@ | |||
# |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
included so we could do anonymous publishing to verdaccio
@@ -18,12 +18,10 @@ function cleanup { | |||
then | |||
lsof -ti tcp:4000 | xargs kill | |||
# Clean up generated dists if run locally | |||
shopt -s globstar extglob |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only intel macs need this, m macs will throw on this and crash, failing to cleanup. now that the majority of the common users of this script are on m macs, I'm removing it
rm -rf packages/**/dist | ||
rm -rf storage/ ~/.config/verdaccio/storage/ $output | ||
if [ "$commit_to_revert" != "HEAD" ]; | ||
then | ||
git tag -d $(git tag -l) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lerna default tagged everything and created a commit, using yarn, we can simplify and not create tags and not create extra commits
# build prod docs with a public url of /reactspectrum/COMMIT_HASH_BEFORE_PUBLISH/verdaccio/docs | ||
PUBLIC_URL=/reactspectrum/`git rev-parse HEAD~1`/verdaccio/docs make website-production | ||
PUBLIC_URL=/reactspectrum/`git rev-parse HEAD~0`/verdaccio/docs make website-production |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is the simplification based on lerna used to create an extra commit with tags, now we don't do that, so we can just use the current commit
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
Build successful! π |
## API Changes
unknown top level export { type: 'any' } |
Closes
Comments inline below
I've left a couple peer dependency resolutions for later because they'll be solved by some other PRs we have.
I still want to replace the rest of patch-package, but that can be done in followup
I've managed to remove a total of 99 unique dependencies, a good chunk of that is lerna related.
β Pull Request Checklist:
π Test Instructions:
π§’ Your Project: