-
Notifications
You must be signed in to change notification settings - Fork 299
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(node): Upgrade node dependency to 12.x #1875
Conversation
This commit also removes a duplicate node-sass version from yarn.lock
Verified that @j3tan has signed the CLA. Thanks for the pull request! |
Also a question on whether we need to upgrade |
@@ -18618,31 +18564,6 @@ node-sass@^4.13.1: | |||
stdout-stream "^1.4.0" | |||
"true-case-path" "^1.0.2" | |||
|
|||
node-sass@^4.9.3: |
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 duplicate yarn.lock
that causes issues if you try to yarn install
on node@12.x
BREAKING CHANGE: Minimum version of node for box-ui-elements bumped to 10.x
c70b81e
to
7a6c641
Compare
@@ -12837,21 +12791,13 @@ fs.realpath@^1.0.0: | |||
resolved "https://registry.yarnpkg.com/fs.realpath/-/fs.realpath-1.0.0.tgz#1504ad2523158caa40db4a2787cb01411994ea4f" | |||
integrity sha1-FQStJSMVjKpA20onh8sBQRmU6k8= | |||
|
|||
fsevents@^1.2.2: |
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.
also had Node 12 issue
* chore(node): Upgrade node dependency to 12.x This commit also removes a duplicate node-sass version from yarn.lock * ci(node): Upgrade node to 12 on CI machines * ci: Remove travis references * docs: Remove travis references * chore(node): Bump minimum node version to 10.x BREAKING CHANGE: Minimum version of node for box-ui-elements bumped to 10.x * chore(node): Removed old version of fsevents from yarn.lock Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
This commit also removes a duplicate node-sass version from yarn.lock
If you run encounter an issue with
yarn install
such as:Verify you have
node@12
installed vianvm
and removenode_modules/sass-variable-parser