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 #685

Closed
wants to merge 4 commits into from
Closed

update dependencies #685

wants to merge 4 commits into from

Conversation

lemastero
Copy link
Contributor

Description

Update library dependencies.
(Current versions of monix and cats have some issues with diverging implicits so they are not updated.)

Proposed Solution

Important Changes Introduced

Testing

Tested by: sbt clean test

Copy link
Contributor

@kapke kapke left a comment

Choose a reason for hiding this comment

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

Remember to update repo.nix files as well, these are like lockfiles from other package managers, but required (and managed) by nix to have build reproducible.
To do that:

  • install nix (may get tricky on macOS Catalina 😅 )
  • enter nix shell (nix-shell)
  • run sbtix-gen-all2 command

build.sbt Outdated
@@ -16,8 +16,8 @@ val commonSettings = Seq(
val malletDeps = Seq(
Dependencies.scopt
).flatten ++ Seq(
"org.jline" % "jline" % "3.1.2",
"net.java.dev.jna" % "jna" % "4.5.1"
"org.jline" % "jline" % "3.16.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

using the occasion: shouldn't jline and jna be picked from project/Dependencies.scala as well?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved to Dependencies.

Copy link
Contributor

@kapke kapke left a comment

Choose a reason for hiding this comment

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

LGTM!
Since this touches metrics, I think @mmrozek should take a look too.

@kapke kapke requested a review from mmrozek September 23, 2020 10:00
@mmrozek
Copy link
Contributor

mmrozek commented Sep 23, 2020

@lemastero you have to sign all commits

@lemastero lemastero mentioned this pull request Sep 23, 2020
@lemastero
Copy link
Contributor Author

Could not sign old commits. I have created different PR: #692

@lemastero lemastero closed this Sep 23, 2020
@lemastero lemastero deleted the update_dependencies branch September 23, 2020 12:32
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.

None yet

3 participants