[Fix] no-arrow-function-lifecycle: prevent crashing on ClassProperty

[AriPerkkio]

Fixes #3093

[codecov-commenter]

Codecov Report

Merging #3098 (c70163d) into master (f93f90a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3098   +/-   ##
=======================================
  Coverage   97.54%   97.54%           
=======================================
  Files         118      118           
  Lines        7855     7857    +2     
  Branches     2801     2802    +1     
=======================================
+ Hits         7662     7664    +2     
  Misses        193      193           

Impacted Files	Coverage Δ
lib/rules/no-arrow-function-lifecycle.js	96.96% <100.00%> (+0.19%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f93f90a...c70163d. Read the comment docs.

[AriPerkkio]

Setting this as draft for now as some old Node + ESLint combinations were failing to parse arrow functions for some reason. Github Action seem to have some trouble running now so cannot verify whether latest commit fixed anything.

[ljharb]

I don't believe we can use the new TS parser on older eslint versions.

[AriPerkkio]

Oh right the ESLint version exclusion is done by the parsers.TS() call. Now that it's used the tests are passing.