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.

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

Fix unused opens false positive for record fields #6846

Merged
merged 2 commits into from
May 24, 2019

Conversation

auduchinok
Copy link
Member

@auduchinok auduchinok commented May 24, 2019

Fixes Unused Open analysis false positive for cases where a record field is used for type inference.

@TIHan TIHan merged commit 0325034 into dotnet:master May 24, 2019
@auduchinok auduchinok deleted the unusedOpens-recordFields branch May 24, 2019 18:16
KevinRansom pushed a commit that referenced this pull request Jun 3, 2019
…6808)

* relax indentations

* fix baseline

* add diagnostics

* add diagnostics

* diagnostics

* diagnostics

* diagnostics

* add diagnostics and possible fix for tp smoke tests

* fix build

* fix build

* more diagnostics

* try to fix flaky test

* Update neg77.fsx

* fix build

* try to fix dodgy test

* Fix unused opens false positive for record fields (#6846)

* Fix unused opens false positive for record fields

* Add record check

* add CI leg to verify assemblies aren't unnecessarily being rebuilt (#6816)

* Removing option from Tuple active pattern (#6772)
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.

3 participants