Skip to content

Add test from PR 50799 for impl PR 56853#57752

Merged
333fred merged 8 commits intodotnet:mainfrom
bernd5:add_test_from_50799_for_56853
Dec 13, 2021
Merged

Add test from PR 50799 for impl PR 56853#57752
333fred merged 8 commits intodotnet:mainfrom
bernd5:add_test_from_50799_for_56853

Conversation

@bernd5
Copy link
Copy Markdown
Contributor

@bernd5 bernd5 commented Nov 13, 2021

With PR #56853 we got support for new lines in string interpolations holes. That PR has some semantic and error tests (for C# < 11) but no syntax test. Therefore I would like to add the test from the closed PR #50799.

@bernd5 bernd5 requested a review from a team as a code owner November 13, 2021 12:43
@ghost ghost added Community The pull request was submitted by a contributor who is not a Microsoft employee. Area-Compilers labels Nov 13, 2021
@bernd5
Copy link
Copy Markdown
Contributor Author

bernd5 commented Nov 15, 2021

The test-error has nothing to do with this PR....

@333fred
Copy link
Copy Markdown
Member

333fred commented Nov 16, 2021

@CyrusNajmabadi @dotnet/roslyn-compiler for another review.

@bernd5
Copy link
Copy Markdown
Contributor Author

bernd5 commented Dec 9, 2021

@CyrusNajmabadi could you have another look?

@333fred 333fred enabled auto-merge (squash) December 9, 2021 15:58
@333fred 333fred added the Test Test failures in roslyn-CI label Dec 9, 2021
@333fred 333fred merged commit 498d5b1 into dotnet:main Dec 13, 2021
@ghost ghost added this to the Next milestone Dec 13, 2021
@333fred
Copy link
Copy Markdown
Member

333fred commented Dec 14, 2021

Thanks @bernd5! Sorry it took a bit to get this merged, we had some infra troubles the past week.

@bernd5 bernd5 deleted the add_test_from_50799_for_56853 branch December 14, 2021 06:17
@Cosifne Cosifne modified the milestones: Next, 17.1.P3 Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers Community The pull request was submitted by a contributor who is not a Microsoft employee. Test Test failures in roslyn-CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants