Add option to the self-closing-comp

[gitim]

This pr adds an option to the self-closing-comp suggested #572, which is corresponding what type of tags ({component: <boolean>, html: <boolean>}) should be self-closed when this is possible. Note: this not changes current behavior, default option is {component: true, html: false}, but in the next major
release we can change it to {component: true, html: true} to enforce all tags to be self-closing when that is possible.

[coveralls]

Coverage increased (+0.005%) to 96.823% when pulling 700dab4 on gitim:fix-self-closing-comp into 750f979 on yannickcr:master.

[coveralls]

Coverage increased (+0.005%) to 96.823% when pulling 52f0f8e on gitim:fix-self-closing-comp into 750f979 on yannickcr:master.

[coveralls]

Coverage increased (+0.005%) to 96.823% when pulling 702570d on gitim:fix-self-closing-comp into 750f979 on yannickcr:master.

[ljharb]

We should indicate which is the default here - "component" i think.

[coveralls]

Coverage increased (+0.005%) to 96.823% when pulling 2978369 on gitim:fix-self-closing-comp into 750f979 on yannickcr:master.

[coveralls]

Coverage increased (+0.005%) to 96.823% when pulling d392381 on gitim:fix-self-closing-comp into 750f979 on yannickcr:master.

[ljharb]

|| true means it will never be false, so i don't think this is good notation. I'd just provide the default object:

"self-closing-comp": ["error", {
  "component": true,
  "html": false
}]

I think it's pretty self-apparent that you can override them with your own booleans

[gitim]

ok, fixed

[coveralls]

Coverage increased (+0.005%) to 96.823% when pulling 74b103d on gitim:fix-self-closing-comp into 750f979 on yannickcr:master.

[ljharb]

Just to be safe, could we also add tests with the options object omitted?

[gitim]

Just to be safe, could we also add tests with the options object omitted?

Yes, sure. Just copied all tests without options and pasted with adding options: [].

[coveralls]

Coverage increased (+0.005%) to 96.823% when pulling 756008b on gitim:fix-self-closing-comp into 750f979 on yannickcr:master.

[ljharb]

LGTM but i'll let @yannickcr do the merging since i'm pretty weak on parser code

[gitim]

@yannickcr, are there any plans to release this fix?

[yannickcr]

@gitim Yes, but I'm currently in holidays and this until the 1st June, then I will be at React Europe on 2-3 June. So do not expect a lot of activity from me until the 4th June.

[yannickcr]

Thanks!