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

c-api: remove #249

Merged
merged 1 commit into from Nov 14, 2022
Merged

c-api: remove #249

merged 1 commit into from Nov 14, 2022

Conversation

kivikakk
Copy link
Owner

Seeing as commonmarker doesn't use the C API to do FFI with comrak any more, I figured I'd remove it — I don't want to maintain it without an actual user, especially given it's C and sharp edges can hurt when forgotten about.

Just wanted to run this by you, @gjtorikian, in case you had other thoughts!

@gjtorikian
Copy link
Collaborator

I completely agree! C API, we hardly knew ye.

@kivikakk kivikakk merged commit 8caec3f into main Nov 14, 2022
@kivikakk kivikakk deleted the remove-c-api branch November 14, 2022 04:01
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.

None yet

2 participants