-
Notifications
You must be signed in to change notification settings - Fork 13
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
initial gitpod configuration #87
base: master
Are you sure you want to change the base?
Conversation
@lefessan What's your take on this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've probably missed many aspects, and don't even know how to test this (yet). But that could be an interesting addition indeed.
opam init | ||
# note autofonce-lib currently needs to be manually installed, see https://github.com/OCamlPro/superbol-studio-oss/issues/79 | ||
opam install autofonce_lib | ||
opam install ./opam/*.opam -y--deps-only |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think here that would now be make dev-deps
in place of the two calls to opam install
.
- name: build superbol-free (LSP) | ||
command: | | ||
gp sync-await opam-prepare | ||
make superbol-free |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now make superbol-lsp-server
While this likely isn't "final" this seems to be the minimal needed setup.
I've started locally but recognized fast, that this won't work as the whole vsix part needs node 14 because of its dependencies (stable Debian distributes node 12, which I even had not installed) and the LSP part seems to need a complete ocaml environment.
Much nicer to have that online available.
I'll likely update that later and possibly add a link to use that to a hacking file (currently working on the README).