Expose available formatters in the Node API #2190

Merged
merged 3 commits into from Dec 11, 2016

Projects

None yet

3 participants

@jwilsson
Contributor

Which issue, if any, is this issue related to?

#2181

Is there anything in the PR that needs further explanation?

Not really, I added a basic test. Let me know if you want something more.

jwilsson added some commits Dec 11, 2016
@jwilsson jwilsson Expose available formatters in the Node API 1ed34bf
@jwilsson jwilsson Add a basic stylelint.formatters test
21cc478
@jeddy3
Member
jeddy3 commented Dec 11, 2016

@jwilsson Thanks!

Do you think it's worth documenting this? Perhaps at the bottom of the Formatters doc within the developer guide:

stylelint.formatters

stylelint's internal formatters are exposed publicly in stylelint.formatters.

@jwilsson
Contributor

@jeddy3 Sounds like a good idea, I'll add it!

@jwilsson jwilsson Add docs for stylelint.formatters
29ad7a9
@jeddy3
jeddy3 approved these changes Dec 11, 2016 View changes

LGTM, thanks!

@davidtheclark
Contributor

Thanks @jwilsson!

@davidtheclark davidtheclark merged commit 3acbb62 into stylelint:master Dec 11, 2016

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@davidtheclark
Contributor

Changelog note added:

  • Added: stylelint.formatters exposed in public Node API (#2190).
@jwilsson jwilsson deleted the unknown repository branch Dec 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment