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

Update dependencies #54

Merged
8 commits merged into from
Mar 16, 2017
Merged

Update dependencies #54

8 commits merged into from
Mar 16, 2017

Conversation

ghost
Copy link

@ghost ghost commented Mar 15, 2017

No description provided.

@ghost ghost requested a review from juanpedromoreno March 15, 2017 10:25
@ghost ghost changed the title Update sbt-catalyst-extras to sbt-org-policies WIP - Update sbt-catalyst-extras to sbt-org-policies Mar 15, 2017
@ghost
Copy link
Author

ghost commented Mar 15, 2017

Addresses #56

@ghost ghost changed the title WIP - Update sbt-catalyst-extras to sbt-org-policies WIP - Update dependencies Mar 15, 2017
Copy link
Member

@juanpedromoreno juanpedromoreno left a comment

Choose a reason for hiding this comment

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

Please, let's get back the sharedPublishSettings. Other than that, LGTM. Thanks.

import PgpKeys.gpgCommand

val dev = Seq(Dev("47 Degrees (twitter: @47deg)", "47 Degrees"))
val gh = GitHubSettings("com.fortysevendeg", "github4s", "47 Degrees", mit)
Copy link
Member

Choose a reason for hiding this comment

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

We still need these settings for releasing purposes. Can we get these back?

build.sbt Outdated
@@ -32,8 +27,7 @@ lazy val buildSettings = Seq(
sharedCommonSettings ++
miscSettings ++
sharedReleaseProcess ++
credentialSettings ++
sharedPublishSettings(gh, dev)
Copy link
Member

Choose a reason for hiding this comment

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

Same here, sharedPublishSettings is needed.

@codecov-io
Copy link

codecov-io commented Mar 16, 2017

Codecov Report

Merging #54 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #54   +/-   ##
=======================================
  Coverage   85.82%   85.82%           
=======================================
  Files          27       27           
  Lines         402      402           
  Branches        3        3           
=======================================
  Hits          345      345           
  Misses         57       57
Impacted Files Coverage Δ
...4s/shared/src/main/scala/github4s/HttpClient.scala 78.57% <ø> (ø)
.../jvm/src/main/scala/github4s/util/URLEncoder.scala 0% <0%> (-100%)
...s/js/src/main/scala/github4s/util/URLEncoder.scala 100% <0%> (+100%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed1fa6d...bb30772. Read the comment docs.

@ghost ghost changed the title WIP - Update dependencies Update dependencies Mar 16, 2017
@ghost
Copy link
Author

ghost commented Mar 16, 2017

@juanpedromoreno just added the publish settings back, let's wait for Travis to give us 💚 and we can merge this.

Copy link
Member

@juanpedromoreno juanpedromoreno left a comment

Choose a reason for hiding this comment

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

LGTM!

@ghost ghost merged commit 67fc3f0 into master Mar 16, 2017
@ghost ghost deleted the update-deps branch March 16, 2017 10:20
This pull request was closed.
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.

2 participants