Skip to content

Fix bazel build errors with clang#510

Merged
iche033 merged 14 commits into
gz-msgs11from
shameek/bzlmod
May 12, 2025
Merged

Fix bazel build errors with clang#510
iche033 merged 14 commits into
gz-msgs11from
shameek/bzlmod

Conversation

@shameekganguly

Copy link
Copy Markdown
Contributor

🦟 Bug fix

Summary

Fixes bazel build failures for clang found with:

$ bazel build ... --action_env=CC=/usr/bin/clang

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
@github-project-automation github-project-automation Bot moved this from Inbox to In review in Core development May 12, 2025
Signed-off-by: Shameek Ganguly <shameek@intrinsic.ai>
@iche033 iche033 merged commit c503904 into gz-msgs11 May 12, 2025
10 checks passed
@iche033 iche033 deleted the shameek/bzlmod branch May 12, 2025 22:25
@github-project-automation github-project-automation Bot moved this from In review to Done in Core development May 12, 2025
@codecov

codecov Bot commented May 13, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (gz-msgs11@1f5f19e). Learn more about missing BASE report.

Additional details and impacted files
@@             Coverage Diff              @@
##             gz-msgs11     #510   +/-   ##
============================================
  Coverage             ?   98.33%           
============================================
  Files                ?       27           
  Lines                ?     1204           
  Branches             ?        0           
============================================
  Hits                 ?     1184           
  Misses               ?       20           
  Partials             ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

🏛️ ionic Gazebo Ionic 🪵 jetty Gazebo Jetty

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants