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.

Sign up for GitHub

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

Jump to bottom

[SonarQube] Configure exclusions #5822

Closed
XiangRongLin opened this issue Mar 14, 2021 · 4 comments
Closed

[SonarQube] Configure exclusions #5822

XiangRongLin opened this issue Mar 14, 2021 · 4 comments
Labels
codequality Improvements to the codebase to improve the code quality meta Related to the project but not strictly to code

Comments

@XiangRongLin
Copy link
Collaborator

XiangRongLin commented Mar 14, 2021 •
edited
Loading

In SonarQube files can be excluded from the analysis. I would propse to exclude following files

  1. xml files
  2. test files
  3. database entities (they all seem to end with Entity or Entry) Thinking again, there definetly are ways to do db entities wrong
  4. html files

It's important to at least exclude xml before the first run. Otherwise the analysis has a bunch of useless files
@XiangRongLin XiangRongLin added meta Related to the project but not strictly to code codequality Improvements to the codebase to improve the code quality labels Mar 14, 2021
@XiangRongLin
Copy link
Collaborator Author

@TobiGr @Stypox what are you thought on this?

@XiangRongLin XiangRongLin mentioned this issue Mar 15, 2021
Closed
5 tasks
@XiangRongLin
Copy link
Collaborator Author

XiangRongLin commented Mar 16, 2021 •
edited
Loading

If no one has any objections until saturday im going with the list in the initial body

@TobiGr TobiGr mentioned this issue Mar 16, 2021
Merged
5 tasks
@TobiGr
Copy link
Member

TobiGr commented Mar 17, 2021

List above looks good to me.

@XiangRongLin
Copy link
Collaborator Author

XiangRongLin commented Mar 20, 2021 •
edited
Loading

Done by adding
**/*.xml and **/*.html to sonar.exclusions

I misunderstood test files, they should not be excluded

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
codequality Improvements to the codebase to improve the code quality meta Related to the project but not strictly to code
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TobiGr @XiangRongLin