Repository navigation
[Feature request] ESLint plugin #183
Description
For the most part this just works, but functions within tagged markup get weird indent assertions, e.g.,
${collection.map((item) => {
const { prop1, prop2 } = workWith(item); // eslint: expected indentation of 4 but found 10 (indent)
return html` // eslint: expected indentation of 4 but found 10 (indent)
<${Widget} prop1=${prop1} prop2=${prop2} />
`;
})} // eslint: expected indentation of 2 but found 8 (indent)
So I'd request an eslint plugin where you could specify the name of the template function, and make the linter comprehend it, or directions to the resources I'd need to implement one.
Activity
Fordi commentedon Oct 22, 2020
I found
eslint-plugin-lit, which works, albeit by disablinglit/binding-positions(👍 ) andlit/no-invalid-html(😢 ). It doesn't fix the indent comprehension, though. If anyone has any further tips, I'd love to hear 'em.developit commentedon Feb 5, 2021
Starting from the Lit plugin is a great idea - I'd begun hacking on something, but quickly realized it was going to take a lot more work than I had time to prototype:
https://gist.github.com/developit/61d009284458ad4d757991e1756b7f41
ruffin-- commentedon Apr 4, 2022
It looks like spread attributes are also a show-stopping issue with eslint.TIL:
GGabrielDev commentedon Jan 1, 2023
This is kinda old, I know, but there's no progress with this? Would be nice to integrate EsLint to my project knowing that it will work out nicely with HTM.
Sadly, I'm still not savvy enough to write my own EsLint plugin :(
mtsknn commentedon Oct 9, 2024
@Fordi Did you manage to find an ESLint plugin for HTM, given that you closed this issue?
Fordi commentedon Oct 11, 2024
No. But the lit plugin with some configuration to allow for htm's wider syntax support has been "good enough" for the last few years. Mostly, I closed it (and a lot of other outstanding issues that don't look like they're getting fixed any time soon) because I use my GitHub issues at my new job to track active work, and this issue is kinda just polluting that.
mtsknn commentedon Oct 11, 2024
That's good to know, thanks!