Installing should package about jscs-config-seegno HOT 5 CLOSED

This is definitely a dependency, not a dev or peer one. I probably think it'd be easier to simply list the methods where they are necessary.

from jscs-config-seegno.

@ruiquelhas can you take of this? https://github.com/seegno/jscs-config-seegno/blame/master/src/rules/require-should-assertion-execution.js#L8 we can either cache this or mark this a dependency instead.

from jscs-config-seegno.

We have the same issue for jscs, although that one is part of the installation instructions. They were peerDependencies before, to avoid this kind of issues (or at least to provide proper installation warnings).

I'm not sure if it's a good idea to move them to the dependencies, so I would suggest to just add should to the installation instructions.

from jscs-config-seegno.

I have a better solution for this I think - invoke the require call inside a try catch and only add this rule if the should package is installed. Since most dev envs will already have it, those who forget to install it or are simply not interested in loading it, will not suffer from it.

from jscs-config-seegno.

Not solved by #23. We still need to implement @ruimarinho suggestion.

from jscs-config-seegno.