Skip to content
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

Add CircleCI build #4

Merged
merged 2 commits into from
Dec 21, 2017
Merged

Add CircleCI build #4

merged 2 commits into from
Dec 21, 2017

Conversation

brendanlong
Copy link
Contributor

No description provided.

@brendanlong brendanlong force-pushed the circleci branch 8 times, most recently from 6cb6cc6 to 12c10de Compare December 21, 2017 14:41
@brendanlong brendanlong self-assigned this Dec 21, 2017
@brendanlong brendanlong requested a review from a team December 21, 2017 15:17
@@ -11,7 +11,7 @@ install: [jbuilder install]
remove: [jbuilder uninstall]

depends: [
"async" {= "v0.9.1"}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

v0.9.1 doesn't exist.. oops.

@@ -1,3 +1,5 @@
[![CircleCI](https://circleci.com/gh/arenadotio/pgx.svg?style=shield)](https://circleci.com/gh/arenadotio/pgx)

Choose a reason for hiding this comment

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

Can remove the comment from line 16 now that this is added.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@@ -11,7 +11,7 @@ install: [jbuilder install]
remove: [jbuilder uninstall]

depends: [
"async" {= "v0.9.1"}

Choose a reason for hiding this comment

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

Is this change relevant to circle ci?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is required because we're using opam install --deps-only pgx_async, and v0.9.1 doesn't exist. I moved the change to a different commit with a description.

v0.9.1 doesn't exist. We didn't run into this internally since we
don't use the .opam files in our repo.
Copy link

@bradlangel bradlangel left a comment

Choose a reason for hiding this comment

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

Lgtm!

@brendanlong brendanlong merged commit b001b77 into master Dec 21, 2017
@brendanlong brendanlong deleted the circleci branch December 21, 2017 15:42
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