Skip to content

Test building python bindings in Ubuntu CI#672

Merged
scpeters merged 1 commit intogz-math8from
scpeters/ci_build_python_bindings
Apr 9, 2025
Merged

Test building python bindings in Ubuntu CI#672
scpeters merged 1 commit intogz-math8from
scpeters/ci_build_python_bindings

Conversation

@scpeters
Copy link
Copy Markdown
Member

@scpeters scpeters commented Apr 9, 2025

🦟 Bug fix

Adds CI testing for feature added in #636

Summary

The ability to build python bindings separately
from the core library was added in #636 and is
currently used when building the homebrew
formulae but is not directly tested in CI.
This adds a quick test to the Ubuntu GitHub
actions workflow to compile the python bindings
after compiling example code.

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.

The ability to build python bindings separately
from the core library was added in #636 and is
currently used when building the homebrew
formulae but is not directly tested in CI.
This adds a quick test to the Ubuntu GitHub
actions workflow to compile the python bindings
after compiling example code.

Signed-off-by: Steve Peters <scpeters@openrobotics.org>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.71%. Comparing base (44e3bba) to head (d3b25f1).
Report is 16 commits behind head on gz-math8.

Additional details and impacted files
@@            Coverage Diff            @@
##           gz-math8     #672   +/-   ##
=========================================
  Coverage     95.71%   95.71%           
=========================================
  Files           150      153    +3     
  Lines         10390    10466   +76     
=========================================
+ Hits           9945    10018   +73     
- Misses          445      448    +3     

☔ 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.

@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Apr 9, 2025
@scpeters scpeters merged commit 664e4f9 into gz-math8 Apr 9, 2025
18 checks passed
@scpeters scpeters deleted the scpeters/ci_build_python_bindings branch April 9, 2025 23:42
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Apr 9, 2025
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.

3 participants