-
Notifications
You must be signed in to change notification settings - Fork 70
update npm versions #255
base: main
Are you sure you want to change the base?
update npm versions #255
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
"devDependencies": { | ||
"autoprefixer": "^9.8.6", | ||
"js-yaml": "^3.14.0", | ||
"lodash": ">=4.17.19", | ||
"mixin-deep": ">=1.3.2", | ||
"postcss-cli": "^6.1.3", | ||
"postcss-cli": "^8.3.1", |
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.
@vaikas did you tested the website locally and there is not problems with new version?
I asked because the bump is a major semantic change which usually comes with braking APIs
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 particular tool is only used for processing CSS before shipping, and should be okay.
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.
It looks like this was somehow confused by the master
/main
transition. Hoping a comment will kick things off the right way.
"devDependencies": { | ||
"autoprefixer": "^9.8.6", | ||
"js-yaml": "^3.14.0", | ||
"lodash": ">=4.17.19", | ||
"mixin-deep": ">=1.3.2", | ||
"postcss-cli": "^6.1.3", | ||
"postcss-cli": "^8.3.1", |
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 particular tool is only used for processing CSS before shipping, and should be okay.
@vaikas: PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Update packages that are out of date.