Skip to content
This repository has been archived by the owner on Jul 30, 2024. It is now read-only.

Update Pekko dependencies #221

Merged
merged 1 commit into from
Mar 29, 2024
Merged

Update Pekko dependencies #221

merged 1 commit into from
Mar 29, 2024

Conversation

mdedetrich
Copy link
Collaborator

No description provided.

Signed-off-by: Matthew de Detrich <[email protected]>
@mdedetrich mdedetrich force-pushed the update-pekko-dependencies branch from c5bf6c1 to 9988fa1 Compare March 23, 2024 08:40
@@ -31,7 +31,7 @@ object Dependencies {
private val scalaParallelCollections =
"org.scala-lang.modules" %% "scala-parallel-collections" % versions.scalaParallelCollections

private val testSeq = Seq(scalaTest, pekkoTestKit)
private val testSeq = Seq(scalaTest, pekkoTestKit, pekkoStream)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is needed otherwise you get

[info] org.zalando.kanadi.CommitCursorOmittedSpec *** ABORTED ***
[info]   java.lang.IllegalStateException: You are using version 1.0.2 of Apache Pekko, but it appears you (perhaps indirectly) also depend on older versions of related artifacts. You can solve this by adding an explicit dependency on version 1.0.2 of the [pekko-protobuf-v3, pekko-stream] artifacts to your project. Here's a complete collection of detected artifacts: (1.0.1, [pekko-protobuf-v3, pekko-stream]), (1.0.2, [pekko-actor, pekko-testkit]). See also: https://pekko.apache.org/docs/pekko/current/common/binary-compatibility-rules.html#mixed-versioning-is-not-allowed

@coveralls
Copy link

Pull Request Test Coverage Report for Build 8400646862

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 74.341%

Totals Coverage Status
Change from base Build 7450681699: 0.0%
Covered Lines: 423
Relevant Lines: 569

💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8400646862

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 74.341%

Totals Coverage Status
Change from base Build 7450681699: 0.0%
Covered Lines: 423
Relevant Lines: 569

💛 - Coveralls

@coveralls
Copy link

coveralls commented Mar 23, 2024

Pull Request Test Coverage Report for Build 8400646817

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 74.341%

Totals Coverage Status
Change from base Build 7450681699: 0.0%
Covered Lines: 423
Relevant Lines: 569

💛 - Coveralls

@gchudnov gchudnov merged commit 06e0df8 into master Mar 29, 2024
7 checks passed
@gchudnov
Copy link
Collaborator

Hi @mdedetrich
thank you for the PR, merged it.

One update: the company has decided to move all nakadi related projects to the internal github-repository and archive the public ones. In this way there won't be any updates anymore...

--
I wanted to thank you for all the hard work and dedication you've shown on the project.
Your commitment to the project is truly inspiring. Thank you!

@mdedetrich
Copy link
Collaborator Author

Hi @mdedetrich
thank you for the PR, merged it.

One update: the company has decided to move all nakadi related projects to the internal github-repository and archive the public ones. In this way there won't be any updates anymore...

--
I wanted to thank you for all the hard work and dedication you've shown on the project.
Your commitment to the project is truly inspiring. Thank you!

Thanks for letting me know! I'm happy that Kanadi was useful until now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants