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

Implement spec for standard mailbox combinations in Akka.NET #1897

Merged
merged 1 commit into from
Apr 21, 2016
Merged

Implement spec for standard mailbox combinations in Akka.NET #1897

merged 1 commit into from
Apr 21, 2016

Conversation

wmeints
Copy link
Contributor

@wmeints wmeints commented Apr 19, 2016

Added some basic specs for the mailbox/actor combinations in Akka.NET.
Bounded dispatcher and mailbox configurations haven't been added yet
as these are not supported right now.

Takes care of issue #1793

@@ -0,0 +1,177 @@
//-----------------------------------------------------------------------
// <copyright file="MailboxesSpec.cs" company="Akka.NET Project">
// Copyright (C) 2009-2016 Typesafe Inc. <http://www.typesafe.com>
Copy link
Contributor

@marcpiechura marcpiechura Apr 21, 2016

Choose a reason for hiding this comment

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

Could you change the name and url from Typesafe to Lightbend?
I already created another PR which does that for all other files.

@marcpiechura
Copy link
Contributor

Rest looks good to me 👍

Added some basic specs for the mailbox/actor combinations in Akka.NET.
Bounded dispatcher and mailbox configurations haven't been added yet
as these are not supported right now.

Takes care of issue #1793
@wmeints
Copy link
Contributor Author

wmeints commented Apr 21, 2016

Fixed the copyright lines in the file. Will update the original issue with the missing mailboxes.

@marcpiechura marcpiechura merged commit bfb2fc8 into akkadotnet:dev Apr 21, 2016
This was referenced Apr 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants