📎 Implement lint/noThisInStatic - mysticatea/no-this-in-static

[Conaclos]

Description

Implement @mysticatea/no-this-in-static.

Want to contribute? Lets we know you are interested! We will assign you to the issue to prevent several people to work on the same issue. Don't worry, we can unassign you later if you are no longer interested in the issue! Read our contributing guide and analyzer contributing guide.

[ditorodev]

I'm gonna try and take a hit on this! Lets see how it goes

[ditorodev]

Update: Currently getting used to it. @Conaclos has pointed me to some good example rules that I could use as reference but this weekend I don't have much time to keep trying. From Wednesday next week ill get a lot more time to try implementing this

Resources

conaclos — 08/30/2023 10:31 PM
You can take a look to an existing rule: https://github.com/biomejs/biome/blob/main/crates/rome_js_analyze/src/analyzers/suspicious/no_duplicate_class_members.rs#L101

conaclos — 08/30/2023 10:31 PM
There is no specific method is_static.
You have to traverse the modifiers of a member and look for the static modifier
Another example: https://github.com/biomejs/biome/blob/main/crates/rome_js_analyze/src/semantic_analyzers/nursery/use_naming_convention.rs#L677

conaclos — 08/30/2023 10:38 PM
You shoudl also take a look at no_setter_return.rs. This could help you 🙂

[ditorodev]

This week im back at actively trying this 🤞

[ditorodev]

Been making amazing progress here, check the PR #366