Skip to content

Remove HIDE_SYMBOLS_BY_DEFAULT: replace by a default configuration in gz-cmake.#408

Merged
mjcarroll merged 1 commit intomainfrom
jrivero/disable_hide_symbols_default
Jan 12, 2024
Merged

Remove HIDE_SYMBOLS_BY_DEFAULT: replace by a default configuration in gz-cmake.#408
mjcarroll merged 1 commit intomainfrom
jrivero/disable_hide_symbols_default

Conversation

@j-rivero
Copy link
Contributor

… gz-cmake.

See gazebosim/gz-cmake#166 (comment)

Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org>
@j-rivero j-rivero requested a review from iche033 as a code owner January 11, 2024 18:31
@github-actions github-actions bot added the 🏛️ ionic Gazebo Ionic label Jan 11, 2024
@codecov
Copy link

codecov bot commented Jan 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b834d76) 72.34% compared to head (174f973) 72.36%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #408      +/-   ##
==========================================
+ Coverage   72.34%   72.36%   +0.01%     
==========================================
  Files          37       37              
  Lines        5052     5055       +3     
==========================================
+ Hits         3655     3658       +3     
  Misses       1397     1397              

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

@mjcarroll mjcarroll merged commit c563fee into main Jan 12, 2024
@mjcarroll mjcarroll deleted the jrivero/disable_hide_symbols_default branch January 12, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏛️ ionic Gazebo Ionic

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants