-
Notifications
You must be signed in to change notification settings - Fork 1
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
Chore(repo): Migration to Yarn Modern #1029
Conversation
✅ Deploy Preview for spirit-design-system-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for spirit-design-system-demo canceled.
|
✅ Deploy Preview for spirit-design-system-validations canceled.
|
839638d
to
446e189
Compare
446e189
to
77de4da
Compare
3d14103
to
8c3bd7e
Compare
2290b1b
to
fadbcb1
Compare
ac62d0f
to
2f02196
Compare
5f0fd0f
to
6ee9d3e
Compare
6ee9d3e
to
34cc809
Compare
✅ Deploy Preview for spirit-design-system ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
1ce0b33
to
3b9782d
Compare
a1da524
to
b91b19f
Compare
021ebac
to
183f7be
Compare
@adamkudrna @crishpeen From my point of view this is read-to-go. Let's discuss the documentation of getting started with Yarn, but I think this is covered by prerequisites in the main README. Please, be short. |
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.
So I tested it and it works.
I just pulled the branch, make install
, make build
and make start
and no problems. I didn't have to do anything more, that's great.
So please just fix those two points I wrote earlier today and it is good to go for me.
Good job!
38bca97
to
56a65f0
Compare
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.
Thank you for cleaning up the task names! 👏🏻
* first parent id given is `undefined` because it is a starting point * use `resolve` plugin to find directory indexes (index.js) * set `preventAssignment` to true because it will be the default in the next Rollup release
* `normalizeColor` can now produce both short and long hex colors base on the flag
* fixed supported node version
* `normalizeColor` can now produce both short and long hex colors base on the flag
56a65f0
to
7e5e179
Compare
Description
Yarn v1 is an old implementation and has been recommended to migrate to modern versions. Today's version is v4 (https://yarnpkg.com/blog/release/4.0) - we cannot go to v4 since it requires Node.js v18.
We are migrating to the latest v3 (https://v3.yarnpkg.com/) which is far more performant and has better verbose messages.
There are a few changes like the configuration going through the
yarnrc.yml
or that thepre
andpost
hooks do not work (due to possible collisions in the script lifecycle).Please, give this everyone a try. I need to know that everything is working for you. And finally we will be a little more up-to-date :-)
Additional context
There were some additional modifications that need to be done:
The changes are in separate commits to allow better fixing.
Issue reference
refs #264
Before submitting the PR, please make sure you do the following
fixes #123
).