search

stylus / nib

Stylus mixins, utilities, components, and gradient image generation
http://stylus.github.io/nib
MIT License
1.91k stars 249 forks source link

Please upgrade stylus dependency version to fix NodeJS 14 warnings #347

Closed AlynxZhou closed 3 years ago

AlynxZhou commented 3 years ago

https://github.com/stylus/stylus/pull/2538 fixed NodeJS 14 warnings for stylus, and a new version of stylus has released. But nib is still using 0.54.5 in its package.json (Here), please upgrade it to 0.54.8 to fix this.

slightlyfaulty commented 3 years ago

There's already #336 for this which would be a better fix going forward.

AlynxZhou commented 3 years ago

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 notifications@github.com wrote:

There's already #336 https://github.com/stylus/nib/pull/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 https://github.com/stylus/nib/issues/347#issuecomment-662932405, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACANVR55R2V4RVVLKL7PMWTR5AGHTANCNFSM4O5K3KLA .

slightlyfaulty commented 3 years ago

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"
}
AlynxZhou commented 3 years ago

That's good, but I am using NPM, and NPM is OK so I have no reason to switch to yarn...

Saul Fautley notifications@github.com 于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 https://github.com/stylus/nib/issues/347#issuecomment-662947214, 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

slightlyfaulty commented 3 years ago

Yes you are right. NPM is ok... Yarn is just much, much better 😏

ADobrianskiy commented 3 years ago

Guys, please, update the stylus version( It is so ugly to see these warnings.

AlynxZhou commented 3 years ago

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.

slightlyfaulty commented 3 years ago

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.