Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audit of implemented rules #34

Open
dedece35 opened this issue Jan 4, 2023 · 1 comment
Open

Audit of implemented rules #34

dedece35 opened this issue Jan 4, 2023 · 1 comment
Labels
🗃️ rule rule improvment or rule development or bug java php __PRIO_LOW__ python

Comments

@dedece35
Copy link
Member

dedece35 commented Jan 4, 2023

Is your feature request related to a problem? Please describe.
There is a lot of implemented rules for each language in ecocode plugin.
Maybe some rules already exist natively in SonarQube.
And second point, check the veracity of each rule :

Describe the solution you'd like
It would be good to make a verification audit to check if :

  • implemented rules doesn't exist in SonarQube native rules yet
  • implemented code of each rule to check that all enabled statement is really managed on casting process ! (see [PRIORITY HIGH] Crash SonarQube analysis #42 )
  • check if native SonarQube rule (ex : "Pattern.compile in static scope instead of method scope") can be tagged (with our common tool in ecoCode-common repository) with eco-design tag

Describe alternatives you've considered
For ambiguous rules (implemented VS SonarQube native), list them and discuss about it during a core team point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗃️ rule rule improvment or rule development or bug java php __PRIO_LOW__ python
Projects
None yet
Development

No branches or pull requests

1 participant