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: fix CI build #240

Merged
merged 1 commit into from Oct 2, 2022
Merged

c-api: fix CI build #240

merged 1 commit into from Oct 2, 2022

Conversation

kivikakk
Copy link
Owner

Broken by 12c32fa.

Of course, it does break commonmarker. Currently got Linux/mac builds OK, but working through Windows whatever: https://github.com/gjtorikian/commonmarker/actions/runs/2857777814

@gjtorikian
Copy link
Collaborator

👋 Could I do anything to help here? I've finally got multiplatform gem compilation working (gjtorikian/commonmarker#189), but the Windows gem doesn't seem to be building. I wonder if it's related to this change?

@kivikakk
Copy link
Owner Author

I'm sure it's related, yeah. Any help you could offer would be appreciated — life's gotten tricky so I might not have time for a couple weeks at least.

@kivikakk
Copy link
Owner Author

kivikakk commented Oct 2, 2022

I'm just going to merge this for now to get comrak's own CI going again so I can assess other PRs.

@kivikakk kivikakk merged commit a9af78b into main Oct 2, 2022
@kivikakk kivikakk deleted the try-fix-ci branch October 2, 2022 03:04
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