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

R client: fix warning caused by not checking return value of function #5382

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

kosak
Copy link
Contributor

@kosak kosak commented Apr 18, 2024

The code is make a call and not checking the return value for an error.

@kosak kosak added NoDocumentationNeeded NoReleaseNotesNeeded No release notes are needed. labels Apr 18, 2024
@kosak kosak self-assigned this Apr 18, 2024
@kosak kosak added this to the 2. April 2024 milestone Apr 18, 2024
@kosak kosak enabled auto-merge (squash) April 18, 2024 14:21
Copy link
Contributor

@alexpeters1208 alexpeters1208 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kosak kosak merged commit d3e932a into deephaven:main Apr 18, 2024
14 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Apr 18, 2024
@kosak kosak deleted the kosak_fix-r-warning branch April 21, 2024 14:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants