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

[biomejs]

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

[ditorodev]

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

[ditorodev]

This week im back at actively trying this 🤞

[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]

Been making amazing progress here, check the PR #366