sinclairnick / brail

💌 Transactional email that feels different
https://brail.dev
Apache License 2.0
258 stars 6 forks source link

Linting: Add linting for Brail React components #15

Open sinclairnick opened 1 year ago

sinclairnick commented 1 year ago

Currently, linting is only applied to HTML components. This is helpful for "off-roading", but there are also lints that would help the more guard-railed @brail/react package. I'm more inclined to use linting over throwing errors, when possible, as it keeps the "correctness" contained within ESLint, instead of having to reinvent the wheel here.

While there may be some overlap with HTML and Brail components, the linting should be hard-separated, since the internals are much different. E.g. with="..." on an HTML element is not equivalent to the same property on a given Brail component. Duplication (which will be minimal) is preferred, when common linting does arise.