-
Notifications
You must be signed in to change notification settings - Fork 130
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
Merlin #233
Merlin #233
Conversation
4f46a0b
to
9165e30
Compare
17fd1b8
to
1ecfbf3
Compare
00f7351
to
19552ad
Compare
For some reason the xml tests are quite unstable and failing randomly. But merlin tests work well. It's conceivable that we can use ocaml binaries rather instead as described here: http://blog.mlin.net/2013/02/testing-ocaml-projects-on-travis-ci.html (though this blog post looks quite old). @magnars, let me know if you are happy to merge as is. |
21e12b4
to
c07b2a2
Compare
cc58c53
to
6b86657
Compare
I switched travis to use binary ocaml. The tests failed but only for latest emacs and xml and I'm pretty sure it's not caused by this PR. @magnars, I hope you're happy to merge this. |
Hey, @magnars, any opinion on this? |
Ping @magnars. |
This wasn't actually merged, was it? |
Ah, I've just noticed that you added a warning that this package is now abandoned. |
#152 but with working tests (except tests are quite broken with emacs 25.2, see #232).