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.

Sign up for GitHub

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

Jump to bottom

Scope_DeclaratorArguments_* tests for Out Variables should be cloned for Pattern Variables. #13660

Closed
AlekseyTs opened this issue Sep 7, 2016 · 2 comments
Closed

Scope_DeclaratorArguments_* tests for Out Variables should be cloned for Pattern Variables. #13660

AlekseyTs opened this issue Sep 7, 2016 · 2 comments
Assignees
@gafter
Labels
Area-Compilers Language-C# New Language Feature - Pattern Matching Pattern Matching Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented Test Test failures in roslyn-CI
Milestone
2.0 (RTM)

Comments

@AlekseyTs
Copy link
Contributor

No description provided.

@AlekseyTs AlekseyTs added this to the 2.0 (Preview 5) milestone Sep 7, 2016
@gafter gafter removed this from the 2.0 (Preview 5) milestone Sep 7, 2016
@gafter gafter removed their assignment Sep 7, 2016
@gafter gafter mentioned this issue Sep 8, 2016
Closed
@gafter
Copy link
Member

gafter commented Sep 8, 2016

These tests are part of fixing the feature; dup of #13460.

@gafter gafter closed this as completed Sep 8, 2016
@gafter
Copy link
Member

gafter commented Sep 8, 2016

Also #13459, which was more specifically about the tests.

@jaredpar jaredpar reopened this Sep 19, 2016
@jaredpar jaredpar added this to the 2.0 (RTM) milestone Sep 19, 2016
@jcouv jcouv added the Test Test failures in roslyn-CI label Jan 17, 2017
gafter added a commit to gafter/roslyn that referenced this issue Jan 27, 2017
@gafter
Duplicate some existing scoping tests for pattern declarations in dec…
fe76c01 
…larator arguments, where they are not permitted.

Fixes dotnet#13660
@gafter gafter mentioned this issue Jan 27, 2017
Merged
@gafter gafter added the 4 - In Review A fix for the issue is submitted for review. label Jan 27, 2017
@gafter gafter added Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented and removed 4 - In Review A fix for the issue is submitted for review. labels Feb 9, 2017
gafter added a commit that referenced this issue Feb 9, 2017
@gafter
Duplicate some existing scoping tests for pattern declarations in dec…
260519d 
…larator arguments, where they are not permitted. (#16804)

* Duplicate some existing scoping tests for pattern declarations in declarator arguments, where they are not permitted.
Fixes #13660
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gafter gafter

Labels
Area-Compilers Language-C# New Language Feature - Pattern Matching Pattern Matching Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented Test Test failures in roslyn-CI
Projects
None yet
Milestone
2.0 (RTM)
Development

No branches or pull requests
4 participants
@jaredpar @gafter @AlekseyTs @jcouv