-
Notifications
You must be signed in to change notification settings - Fork 250
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
Please upgrade stylus dependency version to fix NodeJS 14 warnings #347
Comments
There's already #336 for this which would be a better fix going forward. |
It was submitted 3 years ago and still open, I am not sure when it will be
merged...I want to remove my last NodeJS warning.
…On Thu, Jul 23, 2020, 18:29 Saul Fautley ***@***.***> wrote:
There's already #336 <#336> for this
which would be a better fix going forward.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#347 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACANVR55R2V4RVVLKL7PMWTR5AGHTANCNFSM4O5K3KLA>
.
|
Yeah, I feel you... If you're using Yarn you can always just add this to your package.json in the meantime: "resolutions": {
"nib/stylus": "0.54.8"
} |
That's good, but I am using NPM, and NPM is OK so I have no reason to
switch to yarn...
Saul Fautley <[email protected]> 于2020年7月23日周四 下午7:09写道:
… Yeah, I feel you...
If you're using Yarn you can always just add this to your package.json in
the meantime:
"resolutions": {
"nib/stylus": "0.54.8"
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#347 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACANVR7ZK5DWDAV2DURAXI3R5AK7PANCNFSM4O5K3KLA>
.
--
*Alynx Zhou*
*A Coder & Dreamer*
*Intern in SUSE Beijing*
*Student of **Beijing Jiaotong University*
*School of Computer and Information Technology*
|
Yes you are right. NPM is ok... Yarn is just much, much better 😏 |
Guys, please, update the stylus version( It is so ugly to see these warnings. |
Anyway, I just dropped nib for my blog currently, it looks fine...but only on newer browsers I think, it's acceptable for my personal blog, but not everything with stylus. |
I wouldn't count on nib being updated any time soon. Do yourself a favour and drop nib from your toolbox. You really don't (and shouldn't) need nib in this day and age. Copy the mixins you use to your own lib and use autoprefixer for vendor prefixing. |
stylus/stylus#2538 fixed NodeJS 14 warnings for stylus, and a new version of stylus has released. But nib is still using
0.54.5
in itspackage.json
(Here), please upgrade it to0.54.8
to fix this.The text was updated successfully, but these errors were encountered: