This repository has been archived by the owner on Jun 13, 2020. It is now read-only.

Be nicer changes #36

Open

nkammah wants to merge 3 commits into etsy:master from nkammah:be-nicer

Contributor

nkammah commented Oct 6, 2015

As I'm trying to write a new formatter, I'm suggesting changes on annoying things, like :

some gems are not installed and I really don't care for them for my formatter (eg. elastic search gem)
my formatter is not picked up but I don't know why - list the one you picked up, maybe I had a typo
some options were required when they really could use a default value : when you fail to specify a required option, we don't tell you which one is missing....

nkammah added 3 commits

October 6, 2015 20:57


          move import statements inside class initializer so that nagios herald…

9ccf854

… don't bark when gems are not installed


          list known formatters when you fail to load the requested one

5000c61


          do not require options that can really be optional, it's annoying to …

3df5024

…troubleshoot

nkammah mentioned this pull request

Only works with chef #19

Open

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet