Remove standard-format. fixes #340 #397

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@mafintosh
Collaborator

With this PR standard has around 250 fewer dependencies when installing.

@yoshuawuyts
Collaborator

haha, lgtm

@dcousens
Collaborator
dcousens commented Feb 2, 2016

@mafintosh does this break standard --format?

@feross
Owner
feross commented Feb 3, 2016

@dcousens Yes, it will. We're going to recommend that folks install standard-format separately from now on. The dependencies for standard-format are just too many and it makes install take way too long, even more-so with npm 3. It's not the fault of standard-format. Auto-formatting is complicated and esformatter is just a heavy dependency.

@feross
Owner
feross commented Feb 3, 2016

@mafintosh This looks good, but I want to print a friendlier message to people who run standard --format in the new version. Maybe something like:

Code formatting is available in a standalone package: "standard-format". To install it, run "npm install -g standard-format".
@feross feross added the v6 label Feb 3, 2016
@dcousens
Collaborator
dcousens commented Feb 3, 2016

ACK 👍

@feross
Owner
feross commented Feb 4, 2016

This has been merged into the v6 branch.

@dcousens dcousens closed this Feb 4, 2016
@feross feross referenced this pull request Feb 4, 2016
Closed

Release proposal: standard v6 #399

25 of 25 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment