Skip to content

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