Skip to content

Implement langversion switch for openstaticclasses#7195

Merged
KevinRansom merged 4 commits into
dotnet:release/fsharp47from
KevinRansom:langver-openstaticclasses
Jul 12, 2019
Merged

Implement langversion switch for openstaticclasses#7195
KevinRansom merged 4 commits into
dotnet:release/fsharp47from
KevinRansom:langver-openstaticclasses

Conversation

@KevinRansom

Copy link
Copy Markdown
Contributor
  1. Add switch support to product
  2. Plumb tc globals a bit further down
  3. neg tests for failing to find open static methods
  4. Enable versioning tests

@KevinRansom KevinRansom requested review from TIHan, cartermp and dsyme July 11, 2019 04:50
Comment thread src/fsharp/TypeChecker.fs Outdated
Comment thread src/fsharp/NameResolution.fs Outdated

@TIHan TIHan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, pretty minimal change. Just a quick change requested on the TcGlobals bit which should get rid of some plumbing.

@cartermp cartermp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice and easy, this one

@KevinRansom KevinRansom merged commit 742449a into dotnet:release/fsharp47 Jul 12, 2019
@dsyme

dsyme commented Jul 12, 2019

Copy link
Copy Markdown
Contributor

Great! thanks!

@KevinRansom KevinRansom deleted the langver-openstaticclasses branch August 2, 2019 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants