43081j / eslint-plugin-lit / 9084671957

99%

push

github

fix: allow decorated properties with class fields (#201)

Allows properties decorated with lit property decorators to have class
fields alongside.

This loosens the strictness of the rule under the assumption you have
set `useDefineForClassFields: false` in typescript.

If you use `declare` or `accessor`, those fields will already be ignored
by this rule.

Examples:

```ts
// Error
class X extends LitElement {
  fieldA;
  static properties = {fieldA: {type: String}};
}

// Works now, errored before
class X extends LitElement {
  @property()
  fieldA;
}

// Worked before, works now
class X extends LitElement {
  @property()
  declare fieldA;
}

// Worked before, works now
class X extends LitElement {
  declare fieldA;
  static properties = {fieldA: {type: String}};
}

// Worked before, works now
class X extends LitElement {
  @property()
  accessor fieldA;
}
```

Fixes #193.

573 of 655 branches covered (87.48%)

Branch coverage included in aggregate %.

1029 of 1048 relevant lines covered (98.19%)

48.77 hits per line