Unify metadata files back into one - #203
Conversation
17b50c9 to
a1fdb84
Compare
ElectreAAS
left a comment
There was a problem hiding this comment.
I've taken a look, and while I haven't found any gross mistakes, I gotta say I just don't know enough about the project to take responsibility in approving formally
|
Yes, that's very fair. In an attempt to fill you in a bit: the When I added stable builds I thought it would be necessary to have a different Bundle format for stable builds. But as it turns out it was totally enough to just add an optional |
Sudha247
left a comment
There was a problem hiding this comment.
LGTM!
I have some follow-up questions:
- Do I understand correctly that this is an internal change on the file representations and doesn't affect external (user-facing) endpoints such as
https://get.dune.build/install? - Are there plans to display the stable binaries in the nightly site, with instructions on how to install them?
|
@Sudha247, good questions! To answer them one by one
|
Undoes some of the changes in ocaml-dune#186 be joining the stable metadata into the same file. Closes ocaml-dune#198. Signed-off-by: Marek Kubica <marek@tarides.com>
cdac74f to
fbe1928
Compare
Undoes some of the changes in #186 be joining the stable metadata into the same file.
Closes #198.