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

upgrade dependencies and sbt - scala 3 support #66

Closed
wants to merge 4 commits into from

Conversation

pjfanning
Copy link
Contributor

I started off by seeing if akka-rabbitmq would compile with the newly published akka jars for scala 3 but it looks like there are a number of issues - still thought it would be useful to upgrade some other libs

@pjfanning pjfanning changed the title upgrade dependencies and sbt upgrade dependencies and sbt - scala 3 support Oct 20, 2021
@pjfanning
Copy link
Contributor Author

The latest MR code allows the tests to run with this command:

sbt ++3.1.1-RC1-bin-20211019-7e50e72-NIGHTLY test

Not great for a akka-rabbitmq but progress still.

poohsen pushed a commit that referenced this pull request Nov 3, 2021
* Fully qualified the package directory structures as com.newmotion...
* Merged PR #66 with Scala 3 compatibility changes
* Cleaned up many warning messages that were found with IntelliJ 2021.2
* Updated the Prop constructors
* Updated deprecated setTimer() calls to startSingleTimer() in ConnectionActor.scala

* Upgraded to Scala 2.13.6
* Upgraded to SBT 1.5.5 and cleaned deprecated issues
* Updated to latest dependencies:
    ** ampq-client: 5.9.0 -> 5.13.1
    ** com.typesafe .config: 1.4.0 -> 1.4.1
    ** org.specs2.specs2-mock: 4.10.3 -> 4.13.0

Co-authored-by: PJ Fanning <pjfanning@users.noreply.github.com>
@pjfanning
Copy link
Contributor Author

was merged with #67

@pjfanning pjfanning closed this Nov 3, 2021
@pjfanning pjfanning deleted the upgrade branch November 3, 2021 14:01
@poohsen
Copy link

poohsen commented Nov 3, 2021

@pjfanning I realize now that by squashing PR #67 which contained the changes from this PR I made your contribution less visible (it's in the full text for the commit at the bottom) in the git history for this library. My apologies for that.
I should have squashed & merged this PR first, then request a rebase version of #67 from its author. Next time I will do this right.

@pjfanning
Copy link
Contributor Author

@poohsen no worries - I'm not worried about the fact the 67 merge was squashed - thanks for merging the changes

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

2 participants