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

dev/DeviceResponder: Change debug line to trace #2631

Merged
merged 1 commit into from
Jul 2, 2021
Merged

dev/DeviceResponder: Change debug line to trace #2631

merged 1 commit into from
Jul 2, 2021

Conversation

traversaro
Copy link
Member

YARP 3.4 backport of d02db05 .

Fix #2630 .

@traversaro traversaro requested a review from randaz81 as a code owner July 1, 2021 15:32
@update-docs
Copy link

update-docs bot commented Jul 1, 2021

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update the release notes by adding a file in doc/release/<target_branch>, based on your changes.

@traversaro traversaro requested a review from drdanz July 1, 2021 15:32
@traversaro traversaro mentioned this pull request Jul 1, 2021
Closed
@traversaro traversaro temporarily deployed to code-analysis July 1, 2021 15:34 Inactive
@traversaro traversaro temporarily deployed to code-analysis July 1, 2021 15:34 Inactive
@traversaro traversaro temporarily deployed to code-analysis July 1, 2021 15:34 Inactive
@traversaro
Copy link
Member Author

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update the release notes by adding a file in doc/release/<target_branch>, based on your changes.

The correpsonding fix in master was done without updating the release notes (see d02db05), so I guess the same can be done for this one.

@PeterBowman
Copy link
Member

YARP 3.4 backport

Just in case: please note master branch is set as the target of this PR (instead of yarp-3.4).

@sonarcloud
Copy link

sonarcloud bot commented Jul 1, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@traversaro traversaro changed the base branch from master to yarp-3.4 July 1, 2021 19:02
@traversaro
Copy link
Member Author

YARP 3.4 backport

Just in case: please note master branch is set as the target of this PR (instead of yarp-3.4).

Thanks, this is now fixed.

@drdanz drdanz merged commit 2ffe2b2 into robotology:yarp-3.4 Jul 2, 2021
@traversaro traversaro deleted the fix/2630 branch July 2, 2021 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

@drdanz drdanz Awaiting requested review from drdanz

Assignees

@traversaro traversaro

Labels
Component: Tool - yarprobotinterface Resolution: Merged Target: YARP v3.4.6 Type: Regression This is related to something that worked in a previous version
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

v3.4.3 regression: yarprobotinterface floods its output with Command received and Response sent: messages
3 participants
@traversaro @PeterBowman @drdanz