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

Introduce Deployer as an abstraction #274

Merged
merged 1 commit into from
Nov 14, 2023
Merged

Introduce Deployer as an abstraction #274

merged 1 commit into from
Nov 14, 2023

Conversation

jchapuis
Copy link
Member

Formalize the notion of "deploying a repository" with actor-like semantics.

The deploy functions are simplified, with platform-specific parameters captured by an abstract type. As a result, this is however an API-breaking change.

@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Merging #274 (3cc5fb8) into master (a8c5380) will increase coverage by 0.21%.
The diff coverage is 100.00%.

❗ Current head 3cc5fb8 differs from pull request most recent head d793216. Consider uploading reports for the commit d793216 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #274      +/-   ##
==========================================
+ Coverage   89.93%   90.15%   +0.21%     
==========================================
  Files          71       71              
  Lines         914      914              
  Branches      121       98      -23     
==========================================
+ Hits          822      824       +2     
+ Misses         92       90       -2     
Files Changed Coverage
...deploy/internal/DurableShardedEntityDeployer.scala ø
...deploy/internal/DurableShardedEntityDeployer.scala ø
...ala/endless/runtime/akka/deploy/AkkaDeployer.scala 100.00%
...less/runtime/akka/deploy/AkkaDurableDeployer.scala 100.00%
...y/internal/EventSourcedShardedEntityDeployer.scala 100.00%
...ka/deploy/internal/ShardedRepositoryDeployer.scala 100.00%
.../main/scala/endless/example/app/akka/AkkaApp.scala 100.00%
...ain/scala/endless/example/app/pekko/PekkoApp.scala 100.00%
...a/endless/runtime/pekko/deploy/PekkoDeployer.scala 100.00%
...ss/runtime/pekko/deploy/PekkoDurableDeployer.scala 100.00%
... and 2 more

Formalize the notion of "deploying a repository" with actor-like semantics
@jchapuis jchapuis merged commit 964b3b1 into master Nov 14, 2023
1 check passed
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

1 participant