Skip to content

Test ConvertedType on LHS of deconstruction-assignment#24158

Merged
jcouv merged 2 commits intodotnet:dev15.6.xfrom
jcouv:decon-model
Jan 18, 2018
Merged

Test ConvertedType on LHS of deconstruction-assignment#24158
jcouv merged 2 commits intodotnet:dev15.6.xfrom
jcouv:decon-model

Conversation

@jcouv
Copy link
Copy Markdown
Member

@jcouv jcouv commented Jan 10, 2018

Test-only change following PR feedback (#23975 (review), there was a bug on ConvertedType in foreach-deconstruction).

@AlekseyTs @dotnet/roslyn-compiler for review. Thanks

@jcouv jcouv added Area-Compilers Test Test failures in roslyn-CI labels Jan 10, 2018
@jcouv jcouv added this to the 15.6 milestone Jan 10, 2018
@jcouv jcouv self-assigned this Jan 10, 2018
@jcouv jcouv requested a review from a team as a code owner January 10, 2018 22:05
@AlekseyTs
Copy link
Copy Markdown
Contributor

It doesn't look like any of the scenarios cover deconstruction declaration of the form var (...) = ...

@jcouv
Copy link
Copy Markdown
Member Author

jcouv commented Jan 17, 2018

@AlekseyTs Added test for var (...) =.

Copy link
Copy Markdown
Contributor

@AlekseyTs AlekseyTs left a comment

Choose a reason for hiding this comment

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

LGTM

@jcouv jcouv merged commit ea8aaba into dotnet:dev15.6.x Jan 18, 2018
@jcouv jcouv deleted the decon-model branch January 18, 2018 22:11
@jcouv
Copy link
Copy Markdown
Member Author

jcouv commented Jan 18, 2018

Thanks. I went ahead and merged since test-only change.

333fred added a commit to 333fred/roslyn that referenced this pull request Jan 19, 2018
…g-text

* dotnet/dev15.6.x:
  Fix symbol completion after 'in' (dotnet#24335)
  use PascalCase for const name
  Limit compiler server pipe name length (dotnet#24265)
  Test ConvertedType on LHS of deconstruction-assignment (dotnet#24158)
  remove unused usings
  use .editorconfig files
  address more comments
  address code review comments
  move newly added text into resource file
  add text and hyperlink to C# code style page
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-Compilers Test Test failures in roslyn-CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants