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

Increase code coverage of the CAPI #243

Closed
cschwan opened this issue Sep 7, 2023 · 3 comments
Closed

Increase code coverage of the CAPI #243

cschwan opened this issue Sep 7, 2023 · 3 comments
Assignees
@cschwan

Comments

@cschwan
Copy link
Contributor

cschwan commented Sep 7, 2023

Since #242 was merged we have code coverage of the CAPI, which currently stands at roughly 35%. We should write more C programs using the remaining functions, which could help with #141, too.
@cschwan cschwan self-assigned this Sep 7, 2023
@cschwan
Copy link
Contributor Author

cschwan commented Nov 4, 2023

Commits 965c94a, ed3fc6a, a917fbc after dd327a4, a6aa531, 3511ac8 and 23f130d improve code coverage slightly.

@cschwan
Copy link
Contributor Author

cschwan commented Nov 8, 2023

Commits da6af32, 188a10d, 98b6426, 752a0b2, 7197eda and 2507af8 add more examples, and commits 8d12ba4 and 150a4f7 soft-remove functions that were only used in the exporters/importers that now have long been migrated to the CLI written in Rust.

@cschwan cschwan closed this as completed Nov 8, 2023
@cschwan
Copy link
Contributor Author

cschwan commented Nov 24, 2023

Commit 3b5147f tests a few missing lines and commit 9bbe8d6 removes the obsolete functions completely, as allowed by semver.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cschwan cschwan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cschwan