Fix multi-line lambda coverage regression #1060
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The overlapped lambda accounting is not correct yet...I have some working that fix it but I need to work more on it because there is an ambiguity with C# switch pattern and I'm trying to understand how to solve. |
MarcoRossignoli
added
bug
|
Hi ! Thanks for release, the situation is improving on multi lines lambda 👍 . Coverage is ok when having one lambda on multi line, but when chaining multiple lambdas (for example in Fluent synthax way), coverage has still some holes : One line is OK One lambda chained multiple line is OK Multiple lambdas chained multiple lines is KO |
|
@SebastienPortier can you open another issue pls? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
closes #1056
Before
After
cc: @ulrichb @daveMueller