Skip to content

Don't update beyond scalatest 3.1#1102

Merged
raboof merged 2 commits intoakka:masterfrom
raboof:no-scalatest-3.2
Sep 17, 2020
Merged

Don't update beyond scalatest 3.1#1102
raboof merged 2 commits intoakka:masterfrom
raboof:no-scalatest-3.2

Conversation

@raboof
Copy link
Copy Markdown
Contributor

@raboof raboof commented Aug 18, 2020

Until akka updates, akka/akka-core#29368

Replaces #1094

@raboof raboof requested a review from ignasi35 August 18, 2020 11:23
@raboof
Copy link
Copy Markdown
Contributor Author

raboof commented Aug 18, 2020

Before merging, we should double-check we don't have the problem identified in akka/akka-core#29503

@raboof raboof marked this pull request as draft August 27, 2020 07:08
@raboof raboof mentioned this pull request Aug 27, 2020
@raboof
Copy link
Copy Markdown
Contributor Author

raboof commented Sep 16, 2020

(checked, failures look OK)

@raboof raboof marked this pull request as ready for review September 16, 2020 14:54
@ignasi35
Copy link
Copy Markdown
Contributor

@raboof, akka/akka-core#29503 is still unmerged but I understand it is safe to merge this PR, right? It LGTM and I've approved. Feel free to press the buttons.

@raboof
Copy link
Copy Markdown
Contributor Author

raboof commented Sep 17, 2020

akka/akka#29503 is still unmerged but I understand it is safe to merge this PR, right?

Exactly, this PR makes sure we don't update akka-grpc, which is what we want to do until we upgrade on the Akka side.

@raboof raboof merged commit f0e17a1 into akka:master Sep 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants