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

Stream update 2.4.9 #2335

Merged
merged 22 commits into from
Oct 15, 2016
Merged

Conversation

marcpiechura
Copy link
Contributor

No description provided.

@marcpiechura marcpiechura changed the title [WIP] - Stream update 2.4.9 Stream update 2.4.9 Oct 13, 2016
@marcpiechura marcpiechura removed the WIP label Oct 13, 2016
@@ -15,7 +15,7 @@ namespace Akka.Streams.TestKit.Tests
{
public class TestPublisherSubscriberSpec : AkkaSpec
{
protected readonly ActorMaterializer Materializer;
private ActorMaterializer Materializer { get; }
Copy link
Contributor

Choose a reason for hiding this comment

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

why you need private property?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For running this stream.

@@ -30,7 +30,7 @@ public class InterpreterStressSpec : GraphInterpreterSpecKit
private readonly ITestOutputHelper _helper;

// GraphStages can be reused
private static readonly Select<int,int> Select = new Select<int, int>(x=>x+1, Deciders.StoppingDecider);
private static readonly Select<int,int> Select = new Select<int, int>(x=>x+1);
Copy link
Contributor

Choose a reason for hiding this comment

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

Spaces in lambda

@marcpiechura
Copy link
Contributor Author

@alexvaluyskiy addressed your remarks

@alexvaluyskiy alexvaluyskiy merged commit 4d4c042 into akkadotnet:dev Oct 15, 2016
@Aaronontheweb Aaronontheweb modified the milestone: 1.1.3 Jan 20, 2017
@marcpiechura marcpiechura deleted the stream_update_2.4.9 branch May 17, 2017 19:50
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.

3 participants