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

Merge master to features/source-generators #45921

Merged

Conversation

dotnet-bot
Copy link
Collaborator

This is an automatically generated pull request from master into features/source-generators.

git fetch --all
git checkout merges/master-to-features/source-generators
git reset --hard upstream/features/source-generators
git merge upstream/master
# Fix merge conflicts
git commit
git push upstream merges/master-to-features/source-generators --force

Once all conflicts are resolved and all the tests pass, you are free to merge the pull request.

sharwell and others added 30 commits June 12, 2020 12:33
…c-lambdas

Merge master to features/static-lambdas
…c-lambdas

Merge master to features/static-lambdas
…c-lambdas

Merge master to features/static-lambdas
…c-lambdas

Merge master to features/static-lambdas
…c-lambdas

Merge master to features/static-lambdas
…c-lambdas

Merge master to features/static-lambdas
…c-lambdas

Merge master to features/static-lambdas
Before rewriting the statement, it now checks to see if the Switch expression is being implicitly converted. If so, it explicitly casts the expression to mach the label constant and then rewrites. If not, it rewrites as before.
This will now explicitly cast the switch expression to its converted type every time. If the cast is not needed then the Simplifier will remove the cast.

The cast is now made in the ConvertSwitchStatementToExpressionCodeFixProvider.Rewriter.cs rather than ConvertSwitchStatementToExpressionCodeFixProvider.cs
…c-lambdas

Merge master to features/static-lambdas
Now the ConvertSwitchStatementToExpressionCodeFixProvider.Rewriter.cs applies an explicit cast when it is applicable
AlekseyTs and others added 15 commits July 9, 2020 18:43
…latest design. (#45831)

Related to #45296.

From specification:
If the record is derived from `object`, the record type includes a synthesized readonly property equivalent to a property declared as follows:
```C#
protected Type EqualityContract { get; };
```
The property is `virtual` unless the record type is `sealed`.
The property can be declared explicitly. It is an error if the explicit declaration does not match the expected signature or accessibility, or if the explicit declaration doesn't allow overiding it in a derived type and the record type is not `sealed`.

If the record type is derived from a base record type `Base`, the record type includes a synthesized readonly property equivalent to a property declared as follows:
```C#
protected override Type EqualityContract { get; };
```

The property can be declared explicitly. It is an error if the explicit declaration does not match the expected signature or accessibility, or if the explicit declaration doesn't allow overiding it in a derived type and the record type is not `sealed`. It is an error if either synthesized, or explicitly declared property doesn't override a property with this signature in the record type `Base` (for example, if the property is missing in the `Base`, or sealed, or not virtual, etc.).
The synthesized property returns `typeof(R)` where `R` is the record type.
Remove trailing commas during conversion to class
[Dogfooding] Extend existing IDE code style enforcement rules to EditorFeatures layer
Fix VB case correction for named tuple identifiers
* Use TestHost consistently

* Comments

* Fix

* RenameTestHost

* Renaming
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Auto-approval

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Auto-approval

@JoeRobich JoeRobich merged commit 9d98b0d into features/source-generators Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.