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

Fix warnings #2291

Closed
wants to merge 18 commits into from
Closed

Fix warnings #2291

wants to merge 18 commits into from

Conversation

julianoes
Copy link
Collaborator

Not sure why CI didn't highlight these warnings @bkueng.

Signed-off-by: Julian Oes <julian@oes.ch>
@bkueng
Copy link
Member

bkueng commented Apr 30, 2024

CI is using WERROR=OFF from what I see.

@julianoes
Copy link
Collaborator Author

Oh, you're right. That shouldn't be that way...

@julianoes
Copy link
Collaborator Author

Haha, for now: d3b3c50

Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
@julianoes julianoes changed the title component_metadata: fix unused warnings Fix warnings May 1, 2024
Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
@julianoes julianoes force-pushed the pr-component-metadata-unused branch from 547cb97 to 7eeceaa Compare May 1, 2024 03:31
Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
And rename ubuntu_image to ubuntu_version.

Signed-off-by: Julian Oes <julian@oes.ch>
Signed-off-by: Julian Oes <julian@oes.ch>
@julianoes
Copy link
Collaborator Author

FYI: locally this builds for me with latest MSVC 19.39.

So, hopefully with actions/runner-images#9701 this will pass again.

Copy link

sonarcloud bot commented May 7, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
48.9% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@julianoes
Copy link
Collaborator Author

Closing in favor of #2297.

@julianoes julianoes closed this May 23, 2024
@julianoes julianoes deleted the pr-component-metadata-unused branch May 23, 2024 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants