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

adds a test to PR #19 #20

Merged
merged 4 commits into from
May 11, 2021
Merged

adds a test to PR #19 #20

merged 4 commits into from
May 11, 2021

Conversation

ivg
Copy link
Member

@ivg ivg commented May 7, 2021

I forgot to add so here it is. Most likely it will fail, not because
of the test but because dune will run the test in a wrong package,
where ctoxml is not yet available. But we will see.

ivg added 2 commits May 7, 2021 07:35
I forgot to add so here it is. Most likely it will fail, not because
of the test but because dune will run the test in a wrong package,
where ctoxml is not yet available. But we will see.
@ivg ivg force-pushed the adds-the-missing-test-to-#19 branch from 23cec03 to af7f100 Compare May 7, 2021 21:00
@ivg ivg force-pushed the adds-the-missing-test-to-#19 branch from 8973187 to e8f139a Compare May 11, 2021 21:36
it looks like the cram packages do not work right with opam --with-tests
@ivg ivg force-pushed the adds-the-missing-test-to-#19 branch from e8f139a to 311b5b9 Compare May 11, 2021 21:43
@ivg ivg merged commit 2d56a5f into master May 11, 2021
@ivg ivg deleted the adds-the-missing-test-to-#19 branch May 11, 2021 22: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.

1 participant