Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

suppress test warning in catkin run_tests #448

Merged
merged 2 commits into from
Aug 10, 2023

Conversation

knorth55
Copy link
Member

@knorth55 knorth55 commented May 31, 2022

this PR suppress warning in catkin run_tests.
this trick is introduced in catkin/catkin_tools#405 (comment)

without this PR, travis fails because of long log output.
https://app.travis-ci.com/github/start-jsk/jsk_apc/builds/251296485

@knorth55 knorth55 force-pushed the suppress-warning branch 4 times, most recently from 14dbc95 to 743b00a Compare May 31, 2022 09:35
@knorth55
Copy link
Member Author

kindly ping

pazeshun added a commit to pazeshun/jsk_apc that referenced this pull request May 30, 2023
@pazeshun
Copy link
Collaborator

pazeshun commented Jun 2, 2023

After #455 (comment) is fixed, I want to update this branch (because travis is currently super slow, we should avoid updating until #456 and other follow-up PRs are finished).

@pazeshun
Copy link
Collaborator

@k-okada Could you review this PR?
If this PR is OK, please merge and release it.

@k-okada k-okada merged commit 580056d into jsk-ros-pkg:master Aug 10, 2023
@knorth55 knorth55 deleted the suppress-warning branch August 10, 2023 05:04
@k-okada
Copy link
Member

k-okada commented Aug 10, 2023

pazeshun added a commit to pazeshun/jsk_apc that referenced this pull request Aug 15, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants