We've got some expectations around the styling of Lit syntax. So, to save some time from asking in every patch review, it would be worth enforcing certain style expectations when writing Lit elements. We can do this really easily by installing the [`eslint-plugin-lit`](https://www.npmjs.com/package/eslint-plugin-lit). We already have ESLint installed on the project, so we'd only need to install/enable the package and add the rules we care about. For example, one thing that comes up in patch reviews is to remove the surrounding quotes when declaring properties and attributes in Lit templates. Adding the [quoted-expressions](https://github.com/43081j/eslint-plugin-lit/blob/master/docs/rules/quoted-expressions.md) rule would ensure that this is done before any patch can be approved, without having a reviewer suggesting it in each review.
Bug 1936120 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
We've got some expectations around the styling of Lit syntax. So, to save some time from asking in every patch review, it would be worth enforcing certain style expectations when writing Lit elements. We can do this really easily with the [`eslint-plugin-lit`](https://www.npmjs.com/package/eslint-plugin-lit). We already have ESLint installed on the project, so we'd only need to install/enable the package and add the rules we care about. For example, one thing that comes up in patch reviews is to remove the surrounding quotes when declaring properties and attributes in Lit templates. Adding the [quoted-expressions](https://github.com/43081j/eslint-plugin-lit/blob/master/docs/rules/quoted-expressions.md) rule would ensure that this is done before any patch can be approved, without having a reviewer suggesting it in each review.
We've got some expectations around the styling of Lit syntax. So, to save some time from asking in every patch review, it would be worth enforcing certain style expectations when writing Lit elements. We can do this really easily with [`eslint-plugin-lit`](https://www.npmjs.com/package/eslint-plugin-lit). We already have ESLint installed on the project, so we'd only need to install/enable the package and add the rules we care about. For example, one thing that comes up in patch reviews is to remove the surrounding quotes when declaring properties and attributes in Lit templates. Adding the [quoted-expressions](https://github.com/43081j/eslint-plugin-lit/blob/master/docs/rules/quoted-expressions.md) rule would ensure that this is done before any patch can be approved, without having a reviewer suggesting it in each review.