-
Notifications
You must be signed in to change notification settings - Fork 64
[#210] Initial setup for C language binding #227
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
[#210] Initial setup for C language binding #227
Conversation
@elBoberido I tested it and everything works great. The first step into C land feels great. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #227 +/- ##
==========================================
- Coverage 79.71% 79.27% -0.45%
==========================================
Files 186 190 +4
Lines 20575 20943 +368
==========================================
+ Hits 16401 16602 +201
- Misses 4174 4341 +167
|
13c08d6
to
981c0fa
Compare
774e71d
to
06adfcc
Compare
1bcfc3f
to
9994f7a
Compare
a2c046f
to
9cf49f5
Compare
9cf49f5
to
b989026
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks great out of the box. I reviewed every file again to find something to criticize, but it is perfect.
ec4eeb2
to
b5a18c9
Compare
b5a18c9
to
ec4701c
Compare
Notes for Reviewer
This PR creates the initial setup for the C language bindings
I still need to add some content to the readmes
Pre-Review Checklist for the PR Author
SPDX-License-Identifier: Apache-2.0 OR MIT
iox2-123-introduce-posix-ipc-example
)[#123] Add posix ipc example
)task-list-completed
)Checklist for the PR Reviewer
Post-review Checklist for the PR Author
References
Relates to #210