search

wilson428 / node-lessify

LESS precompiler plugin for Browserify v2
MIT License
24 stars 13 forks source link

update dependencies; #27

Closed Fazod closed 3 years ago

Fazod commented 4 years ago

This updates the LESS version so the vulnerabilities are fixed

DXCanas commented 3 years ago

Sorry, hadn't seen this and don't know what's broken Travis. Will merge once I can verify the update locally!

Fazod commented 3 years ago

Hey @DXCanas - did you already have time to check this out?

DXCanas commented 3 years ago

Wow, thanks for the bump.

I haven't. Making time this week. Sorry about the delay.

DXCanas commented 3 years ago

Actually decided to take a quick look right now.

Given that the Travis issue mostly revolves around a "strict mode" violation to a test in the less package, I'm leaning towards there being an issue with this newer version of less and the version of node that runs the tests.

Can't spend much more time on this right now, but I'll update if I find anything else.

DXCanas commented 3 years ago

@Fazod

Took me a while, but I've bumped the required version of Node to match the newer version of Less you specced.

Will tag and release new version.

DXCanas commented 3 years ago

New version is published, and you've been credited int he readme.

Thanks for the contribution, sorry again about the insane delay.

Fazod commented 3 years ago

Thanks so much David - this is great news and no worries, worth the wait :)