-
Notifications
You must be signed in to change notification settings - Fork 20
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
doc: release procedure #971
base: main
Are you sure you want to change the base?
Conversation
63506ef
to
6a21846
Compare
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.
Instructions for ocaml seem to be missing. Otherwise lgtm
|
||
## OCaml | ||
|
||
There is only the package `hax-engine`, that includes a binary and a |
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.
How does publishing work here? Also, did you want to publish this to opam? I don't see it there.
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 pushed a note, where I explain what we discussed (pinning git basically)
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.
If the note looks good to you, let's resolve this discussion and merge the 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.
lgtm
Please add a note how to update the README if needed, or how the release tag get's on there. Since we don't push it anywhere else, that's essentially the release then and we should describe here how to do that.
Then you can resolve this and merge.
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.
makes sense, started something in branch document-stable-install
bc06da5
to
8d58752
Compare
Co-authored-by: Franziskus Kiefer <[email protected]>
c8522bf
to
4f95210
Compare
This PR adds some documentation regarding the procedure for publishing hax.
It also tweaks some settings for
cargo release
: