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.

Sign up for GitHub

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

Jump to bottom

ign -> gz Upstream Macro Migration : gz-msgs #260

Merged
merged 12 commits into from
Jun 17, 2022
Merged

ign -> gz Upstream Macro Migration : gz-msgs #260

merged 12 commits into from
Jun 17, 2022

Conversation

methylDragon
Copy link
Contributor

@github-actions github-actions bot added the 🌱 garden Ignition Garden label Jun 6, 2022
@methylDragon methylDragon mentioned this pull request Jun 6, 2022
Closed
@chapulina chapulina added the ign to gz Renaming Ignition to Gazebo. label Jun 6, 2022
@chapulina chapulina self-requested a review June 6, 2022 23:54
@chapulina chapulina changed the base branch from main to env_var_migration June 7, 2022 02:01
chapulina
chapulina approved these changes Jun 7, 2022
View reviewed changes
Copy link
Contributor

@chapulina chapulina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once upstream is merged.

@methylDragon methylDragon force-pushed the macro_migration branch 2 times, most recently from 2f1c5ae to f082c02 Compare June 7, 2022 21:28
Base automatically changed from env_var_migration to main June 8, 2022 04:03
chapulina
chapulina reviewed Jun 8, 2022
View reviewed changes
src/ign.hh Outdated Show resolved Hide resolved
@chapulina chapulina added the needs upstream release Blocked by a release of an upstream library label Jun 15, 2022
methylDragon and others added 12 commits June 17, 2022 11:20
@methylDragon
Ticktock IGN_DESCRIPTOR_PATH
3a28285 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Try setting IGN_DESCRIPTOR_PATH to fix coverage
ae4a6aa 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Remove redundant Factory test lines
fe45595 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Ticktock IGN_DESCRIPTOR_PATH
0b89340 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Try setting IGN_DESCRIPTOR_PATH to fix coverage
1c88c6f 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Migrate upstream macros
6f350e0 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Fold in some unrelated comment changes
65a0f90 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Fix an Gazebo to a Gazebo
66681f0 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Tick-tock header functions
6b91dac 
Signed-off-by: methylDragon <[email protected]>
@methylDragon
Fix header function tick-tocks
90e59df 
Signed-off-by: methylDragon <[email protected]>
@chapulina @methylDragon
remove tick-tock of private function
dc33b80 
Signed-off-by: Louise Poubel <[email protected]>
@chapulina @methylDragon
remove outdated migration entry and redundant test
a9f55c6 
Signed-off-by: Louise Poubel <[email protected]>
@methylDragon
Copy link
Contributor Author

@osrf-jenkins run tests please!

@codecov
Copy link

codecov bot commented Jun 17, 2022 •
edited
Loading

Codecov Report

Merging #260 (a9f55c6) into main (e240db2) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #260   +/-   ##
=======================================
  Coverage   14.53%   14.53%           
=======================================
  Files         200      200           
  Lines       12505    12505           
=======================================
  Hits         1817     1817           
  Misses      10688    10688
Impacted Files Coverage Δ
include/gz/msgs/Factory.hh 100.00% <ø> (ø)
src/Generator.cc 92.53% <100.00%> (ø)
src/Utility.cc 82.28% <100.00%> (ø)
src/ign.cc 90.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e240db2...a9f55c6. Read the comment docs.

@methylDragon methylDragon merged commit f82e2e5 into main Jun 17, 2022
@methylDragon methylDragon deleted the macro_migration branch June 17, 2022 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chapulina chapulina chapulina approved these changes

@caguero caguero Awaiting requested review from caguero caguero is a code owner

Assignees
No one assigned
Labels
🌱 garden Ignition Garden ign to gz Renaming Ignition to Gazebo. needs upstream release Blocked by a release of an upstream library
Projects
Core development
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@methylDragon @chapulina