-
Notifications
You must be signed in to change notification settings - Fork 91
WaitForTopics: get content of messages for each topic (backport #353)
#389
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
Conversation
42beec7 to
f330bd9
Compare
adityapande-1995
left a comment
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.
LGTM with full green CI
There were conflicts. Do you want me to push a commit to resolve them? |
|
Yup, that would be great, thanks ! |
|
Sorry, I got sidetracked. Then, I tried to run the release myself and ran into this bug in So, I compiled bloom and ament_lint from source, and then ran into this issue. Regardless, here's the fixes. I can't push to the branch, so I made a PR. I had to use docker to test it because host OS has different behavior than |
* Get content of topic from launch_testing_ros::WaitForTopics #346 Signed-off-by: Pintaudi Giorgio <[email protected]> --------- Signed-off-by: Pintaudi Giorgio <[email protected]> (cherry picked from commit 2d125a5) # Conflicts: # launch_testing_ros/launch_testing_ros/wait_for_topics.py # launch_testing_ros/test/examples/wait_for_topic_launch_test.py Signed-off-by: Aditya Pande <[email protected]>
Signed-off-by: Ryan Friedman <[email protected]> Signed-off-by: Aditya Pande <[email protected]>
cadfbab to
365f968
Compare
No, that shouldn't be the case. We always use the debian packages for the tests, which is version 4.0.1. And looking at the test failures, they aren't failing because of the flake8 version. Something else is going on here. |
Should I make a new ticket? |
I don't think so; whatever is happening here seems to be because of this PR. |
Interesting idea. Not sure if you saw, but I did get |
Oh, that's because this was run with the rolling |
Can anyone do this? I don't see how, and it would be great to get this in. |
This is an automatic backport of pull request #353 done by Mergify.
Cherry-pick of 2d125a5 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/github/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally
Mergify commands and options
More conditions and actions can be found in the documentation.
You can also trigger Mergify actions by commenting on this pull request:
@Mergifyio refreshwill re-evaluate the rules@Mergifyio rebasewill rebase this PR on its base branch@Mergifyio updatewill merge the base branch into this PR@Mergifyio backport <destination>will backport this PR on<destination>branchAdditionally, on Mergify dashboard you can:
Finally, you can contact us on https://mergify.com