Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Upgrade packages that use old versions of graceful-fs #10

Merged
merged 5 commits into from
Aug 1, 2016

Conversation

thomasjo
Copy link
Contributor

The main upgrades here are the various Grunt packages and Less.js, all of which relied on deprecated versions of graceful-fs. Node.js v6 and later requires graceful-fs v4 (or newer).

I also bumped the other packages, and alphabetized the package lists in package.json.

This PR is required as part of atom/atom#12176.
/cc @atom/feedback @kevinsawicki @nathansobo

thomasjo added 5 commits July 15, 2016 18:07
There have been fairly significant changes to the Less.js code since the
previously used version, so this commit was non-trivial to develop. Notably
the use of `Parser` was deprecated and now requires the use of `less.render`
instead.
@winstliu
Copy link

winstliu commented Jul 17, 2016

Closes #8
Closes #7
And maybe even #6

@thomasjo
Copy link
Contributor Author

Closes #8

I completely missed that one... could have saved myself a few hours 😬

@nathansobo nathansobo merged commit 04bdfca into atom:master Aug 1, 2016
@nathansobo
Copy link
Contributor

Thanks @thomasjo!

This was referenced Aug 17, 2016
@silvestreh
Copy link

silvestreh commented Mar 10, 2017

@thomasjo any clues on why @import (optional) isn't available in the included Less@2.7.1? According to Less' documentation, the optional keyword was added in 2.3.0 😕

Edit: Nevermind. I just upgraded to Atom 1.15 and seems to be working!

Another edit: yeah, no. It's not working. I was out of dev mode after restarting Atom and didn't notice.

@winstliu
Copy link

If you're trying to use optional in an Atom package, it's because less-cache was never upgraded in Atom. It was upgraded recently though so you should be able to start using it in Atom 1.17.0 or 1.16.0.

@silvestreh
Copy link

silvestreh commented Mar 10, 2017

@50Wliu yep, I was out of dev mode after updating Atom and didn't notice. That's why I wasn't getting compile errors when modifying my source code. Thanks for pointing it out! 😅

Also, it doesn't seem to be in included 1.16.0-beta0 😞

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants