feat(html-self-closing): add configuration presets

[marekvospel]

see #190 (comment)

Changes

html-self-closing

added preset configuration option

'svelte/html-self-closing': ['warn', {
  'void': 'always',
  // ...
}]
// or
'svelte/html-self-closing': ['warn', 'all']

current presets:

• none: all set to never
• html: html-compliant (always for voit and svelte* elements, others never)
• all: all on always

* to be discussed whether svelte elements should or shouldn't be self closing in html-compliant preset.

Progress

this PR is still a draft, html-self-closing tests are still using the old configuration, no preset tests have been added and the documentation for html-self-closing rule is also outdates

[marekvospel]

*sorry for the amount of commits this PR is going to have. I'm in an environment where I can't install packages, so I can't use type checking, code formatting etc.

[marekvospel]

@ota-meshi @JounQin could you please generate errors.json & output.svelte for the preset tests? I won't be able to do so until saturday / sunday. I've checked allow commits by maintainers, so you should have permissions to make changes in my fork

[JounQin]

I don't have time right now, I'd like to try when I'm free. 🍺

[marekvospel]

The PR is ready! Once the invalid test outputs are generated and verified, it can be merged!

[changeset-bot]

🦋 Changeset detected

Latest commit: 88ea618

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
eslint-plugin-svelte	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[lgtm-com]

This pull request introduces 1 alert when merging 49da137 into 931c062 - view on LGTM.com

new alerts:

• 1 for Property access on null or undefined

[ota-meshi]

We are about to introduce changeset and lgtm into this repository, but have not configured it yet. Ignore these messages for now.
#166

[marekvospel]

I've updated the rule to take either a preset enum or config object, which makes this PR non-breaking

[JounQin]

I've updated the rule to take either a preset enum or config object, which makes this PR non-breaking

Right, so the PR description should be updated. 😁

[marekvospel]

@ota-meshi @JounQin what do you think about the html preset? Should svelte special components like svelte:head be self closing?

[JounQin]

@ota-meshi @JounQin what do you think about the html preset? Should svelte special components like svelte:head be self closing?

According to Angular's HTML parser playground, <svelte:head /> is html compliant while <svelte-head /> is not, so yes.

[marekvospel]

I've realised the test outputs the errors, so I've copied it into the test output files, so the tests should pass now

[marekvospel]

so many typos lmao 😂

[JounQin]

Thanks for your efforts!

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

[ota-meshi]

LGTM! Thank you!