Skip to content

Conversation

@tmcgilchrist
Copy link
Member

No description provided.

@tmcgilchrist tmcgilchrist requested a review from maiste November 8, 2022 07:01
@@ -1,3 +1,3 @@
pin-depends: [
["obuilder-spec.dev" "git+https://github.com/ocurrent/obuilder#a2f4cec1774967a54808e6d5c9dfb6a4d77bc9c8"]
["obuilder-spec.dev" "git+https://github.com/ocurrent/obuilder#8dbe2dd66b344198c9709a3432ed55ba8a5cb113"]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can point to ocaml/opam-repository#22467 once it's released.

Copy link
Member

@maiste maiste left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍
Shouldn't we wait for the PR to be merged in opam before merging?

@maiste
Copy link
Member

maiste commented Nov 8, 2022

I think we can merge it even if the macOS worker is not ok as we are going to run it on a Linux server :)

@TheLortex TheLortex merged commit 7d4630f into main Nov 8, 2022
@MisterDA MisterDA deleted the update_dependencies branch November 8, 2022 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants