-
Notifications
You must be signed in to change notification settings - Fork 128
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
Update gtest to 1.10.0 version #161
Update gtest to 1.10.0 version #161
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me with green CI and once #158 is merged.
aece008
to
c5ab9ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI looks good to me, so approving. What I'll suggest to get this in is:
- Merge Unify master and ros2 branches (merge ros2 into master) #158
- Rebase this one on top of master
- Merge this one
- Update https://github.com/ros2/ros2/blob/master/ros2.repos to point to the master branch here.
- Tag this repository and do a source release. I'll suggest bumping the major version to 3.0.
- Do a release of this package into Rolling.
Signed-off-by: Jose Luis Rivero <jrivero@osrfoundation.org>
c5ab9ab
to
692a0b8
Compare
All right, I've approved, CI is green, and I verified that the changes before and after the rebase are 100% the same. So this is good to go, I'm going to go ahead and merge. Thanks @j-rivero . |
In ros/urdfdom#158 and ros/urdfdom#161, we merged the ros2 branch onto the master branch so there would be single, unified release branch for all of urdfdom. Now that that is done, we should switch to using the master branch of urdfdom.
In ros/urdfdom#158 and ros/urdfdom#161, we merged the ros2 branch onto the master branch so there would be single, unified release branch for all of urdfdom. Now that that is done, we should switch to using the master branch of urdfdom.
Following #158 (review)
Two commits on top of pull #158:
comparison of integer expressions of different signedness