Katherine Scott
Katherine Scott
@dheerubhai-101 by all means! Give it a shot!
@j-rivero can you assign this to @dheerubhai-101?
Sick, should we do a ROS Discourse post on this one and hype it up out on the social media?
@ahcorde, heads up.
@anshium How are things going here? Do you need some help?
@mjcarroll @azeey Can we get this merged? The user has already bumped us twice and has been waiting patiently. The only thing missing AFAICT is approval from @mjcarroll.
I hit the CI for you and did a cursory review. I didn't see anything wrong with the update itself. One thing that I think might need discussion is the...
Looks like everything passes and all requests have been addressed. I'll go ahead and merge this. Thanks for being patient while we were all at ROSCon.
@EliottFerry can you clarify what you mean by, "match previous AddThreeInts implementation."
@anshium there are just a couple of small change requests. Could you please address them so we can get this merged? 😄