-
Notifications
You must be signed in to change notification settings - Fork 692
feat: add opam support #1298
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
feat: add opam support #1298
Conversation
direnv is mentioned in the official OCaml documentation, so having a formal way for direnv to layout does not seem like a bad idea. I often have projects that use OCaml with Node.js, and I just wanted to use |
sounds good! |
Gentle ping |
Signed-off-by: Sora Morimoto <sora@morimoto.io> Co-authored-by: Jonas Chevalier <zimbatm@zimbatm.com>
5f3fcf7
to
5a1be50
Compare
If opam switch is not initialized, this error will happen, but the OPAMSWITCH variable is exposed, so opam will definitely not point to any different switch and will not cause unexpected behavior.
|
@zimbatm Ready to merge! |
@zimbatm Thank you for your review and the effort you put into creating and maintaining this software! |
@zimbatm Could you create a new release for this PR? |
Gentle reminder |
This PR adds support for opam.