Make recommended: true by default and fix inheritence

Updated 2019-10-29: Replaced with content from @marbemac’s comment, which will be used to drive this issue.

recommended should be the default across the board - cli and ruleset extension.

E.g. this:

extends: spectral:oas3

Is the same as:

extends: [[spectral:oas3, recommended]]

As far as extending w all, recommended, or none, it should bubble up the hierarchy (I thought that this was happening now, but maybe not?):

E.g. this:

extends: [[foo.json, all]]

Is the same as:

extends: [[bar.json, all], [baz.json, all]]

Todo

• change recommended to be true by default
• make sure all rules maintain their recommended status (no accidental changes)
• audit documentation to make sure there is no statements which disagree (for JS, CLI, rulesets in general, custom functions, etc)

[marbemac]

My two cents… recommended should be the default across the board - cli and ruleset extension.

E.g. this:

extends: spectral:oas3

Is the same as:

extends: [[spectral:oas3, recommended]]

As far as extending w all, recommended, or none, it should bubble up the hierarchy (I thought that this was happening now, but maybe not?):

E.g. this:

extends: [[spectral:oas3, all]]

Is the same as:

extends: [[spectral:oas3, all], [spectral:oas, all]]

[marbemac]

And perhaps a look at the docs to make sure this behavior is described clearly? @philsturgeon maybe you can take this issue and make it about verification that this is happening + docs audit?