-
Notifications
You must be signed in to change notification settings - Fork 27
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
GitHub Actions #79
GitHub Actions #79
Conversation
Given the nature of the project, I'd err on the side of caution and test as many ocaml versions (major.minor.) as possible unless there's a major cost to doing that. |
I have no idea about the Windows setup. It sounds like a great thing to have in theory. I just don't have the experience running anything on Windows. |
I wondered if the Windows error was caused by Git's CRLF configuration, so I set
@avsm any instructions? |
BTW, @liyishuai, we probably need to do this for everything in |
The current script is quite generic up to installing the dependencies. Other projects can copy these lines and add their own build/test/deploy afterwards. See also ocaml/setup-ocaml#117 |
Update: the script now tests packages that depend on cppo. |
Ready to merge. A few issues to address later:
|
Co-authored-by: Erik Martin-Dorel <[email protected]>
@liyishuai I think you should feel free to merge once you feel you're ready. And again, if you wanted to do the same (very generic) thing for other projects in ocaml-community, we would all be grateful. |
Yes I'll propagate the self-test part to other projects. |
Replacing Travis CI, which is no longer free.
The current workflow has lots of jobs. Maybe worth trimming down a few.
Tests failed on Windows.