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.

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

Added check to ensure receive timeout isn't overwritten #171

Merged
merged 2 commits into from Aug 29, 2023

Conversation

urmahp
Copy link
Collaborator

@urmahp urmahp commented Aug 29, 2023

Added a test to verify the changes

This will fix #165

@codecov
Copy link

codecov bot commented Aug 29, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.06% 🎉

Comparison is base (e85b3ae) 70.35% compared to head (9017a14) 70.42%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #171      +/-   ##
==========================================
+ Coverage   70.35%   70.42%   +0.06%     
==========================================
  Files          69       69              
  Lines        2547     2553       +6     
  Branches      324      325       +1     
==========================================
+ Hits         1792     1798       +6     
  Misses        568      568              
  Partials      187      187              
Files Changed Coverage Δ
include/ur_client_library/ur/dashboard_client.h 100.00% <ø> (ø)
src/ur/dashboard_client.cpp 68.48% <100.00%> (+0.75%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@fmauch fmauch left a 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. Thank you @urmahp !

@fmauch fmauch added this to the Release 1.3.3 milestone Aug 29, 2023
@fmauch fmauch merged commit 647676d into UniversalRobots:master Aug 29, 2023
18 of 19 checks passed
@urmahp urmahp deleted the receive_timeout branch August 30, 2023 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting receive timeout before calling connect() in the dashboard client is not possible.
2 participants