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

Integrate SonarQube or SonarLinting #295

Closed
4 tasks
CMCDragonkai opened this issue Nov 25, 2021 · 1 comment
Closed
4 tasks

Integrate SonarQube or SonarLinting #295

CMCDragonkai opened this issue Nov 25, 2021 · 1 comment
Labels
procedure Action that must be executed

Comments

@CMCDragonkai
Copy link
Member

SonarQube and associated projects provide even higher level linting rules that can help improve team's code quality.

Their list of rules they can apply are here: https://rules.sonarsource.com/typescript

They have a cloud version of their product here: https://sonarcloud.io/welcome. It is free for open source projects.

There seems to be multiple ways of integrating it:

Need to investigate which way is the best.

It seems Sonar would run as part of the CI/CD, separate cloud service, or integrated into our eslint.

Asked a question about this here: un-ts/eslint-plugin-sonar#31

Tasks

  1. - Find out the best way to integrate
  2. - Attempt integration
  3. - Ensure no conflicts with existing eslint issues
  4. - Fix up linting problems!
@CMCDragonkai CMCDragonkai added the procedure Action that must be executed label Nov 25, 2021
@CMCDragonkai
Copy link
Member Author

Our level of linting seems sufficient now. Especially in relation to promise mis-use. So I think we can proceed without this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
procedure Action that must be executed
Development

No branches or pull requests

1 participant