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

cxx compiler report - code issue in sonar qube #981

Closed
iamthegD opened this issue Oct 17, 2016 · 3 comments
Closed

cxx compiler report - code issue in sonar qube #981

iamthegD opened this issue Oct 17, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@iamthegD
Copy link

Hi,

I successfully setup the warning eater from my msbuild log file and I have a question regarding the code issues its supposed/should create ?

In my build log I see : C++ compiler Warnings processed = 2
Which leads me to think I should see 2 code smell or something along those lines.
I'm I missing something or its how should it work ?

Excuse if the format/pertinence is not great, I am not too familiar with github. If this is not appropriate feel free to delete/send to good place.

Thanks for your time.

@guwirth
Copy link
Collaborator

guwirth commented Oct 18, 2016

Hi @iamthegD, looking at the code:

LOG.debug("{} processed = {}", metric == null ? "Issues" : metric.getName(),
it is the number of issues/violations in the report file. If the issues/violations in the report file are invalid (wrong filename, wrong line number, ...) it could be that you can't see them in SQ (see also https://github.com/SonarOpenCommunity/sonar-cxx/wiki/Path-and-path-separator-issues).

@guwirth guwirth added this to the 0.9.7 milestone Oct 18, 2016
@guwirth guwirth self-assigned this Oct 18, 2016
@guwirth
Copy link
Collaborator

guwirth commented Oct 18, 2016

But looking a little bit closer to this I think the info is too early in the code:

LOG.info("{} processed = {}", metric == null ? "Issues" : metric.getName(),
, should be after ...save().

@jmecosta maybe you can move it together with #758.

@iamthegD
Copy link
Author

thanks for the quick info.

Turns out the specific warning I introduced in my code were not turned on by default in the quality gates.

For other who will experience the same problem, look for the C#### values you need ahead of time :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants