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

Fix name conflicts (C) #953

Merged
merged 2 commits into from
May 9, 2021
Merged

Conversation

kovdan01
Copy link

@kovdan01 kovdan01 commented May 8, 2021

Same as #952, but for C. See #952 for details.

@codecov-commenter
Copy link

Codecov Report

Merging #953 (6b4a7b3) into c_master (825017a) will not change coverage.
The diff coverage is 57.14%.

@@            Coverage Diff            @@
##           c_master     #953   +/-   ##
=========================================
  Coverage     55.45%   55.45%           
=========================================
  Files             8        8           
  Lines          1044     1044           
=========================================
  Hits            579      579           
  Misses          465      465           

@redboltz
Copy link
Contributor

redboltz commented May 9, 2021

LGTM, merged.

@redboltz redboltz merged commit 5d30e42 into msgpack:c_master May 9, 2021
@redboltz redboltz mentioned this pull request May 9, 2021
@kovdan01 kovdan01 deleted the fix_name_conflicts_c branch May 10, 2021 14:04
@redboltz redboltz added the C label Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants