Skip to content

Set up opam workflow#21

Merged
chenglou merged 13 commits intomasterfrom
mb-opam
Apr 7, 2017
Merged

Set up opam workflow#21
chenglou merged 13 commits intomasterfrom
mb-opam

Conversation

@tekknolagi
Copy link
Copy Markdown
Contributor

@tekknolagi tekknolagi commented Mar 31, 2017

  • Makefile & topkg
  • Docs
  • CI
  • Update docs/ folder

@tekknolagi tekknolagi requested review from jordwalke and yunxing March 31, 2017 01:09
Comment thread opam Outdated
]
depends: [
"topkg" {>= "0.8.1" & < "0.9"}
"reason-parser" {= "1.13.3"}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should depend on Reason?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yep, sorry

@yunxing
Copy link
Copy Markdown
Contributor

yunxing commented Mar 31, 2017

Please update the CI to use opam, thanks!

@tekknolagi
Copy link
Copy Markdown
Contributor Author

Why does this not install opam 1.2 like the Reason CircleCI build? Who knows????

@tekknolagi
Copy link
Copy Markdown
Contributor Author

@yunxing CircleCI passes. Travis fails because bad config. Let's see if I fixed it...

@chenglou chenglou merged commit 028d1f6 into master Apr 7, 2017
@chenglou chenglou deleted the mb-opam branch April 7, 2017 00: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.

3 participants