Skip to content

EnC emit tests for C# 8.0 patterns#37025

Merged
gafter merged 6 commits intodotnet:masterfrom
ivanbasov:localslotpattern
Jul 11, 2019
Merged

EnC emit tests for C# 8.0 patterns#37025
gafter merged 6 commits intodotnet:masterfrom
ivanbasov:localslotpattern

Conversation

@ivanbasov
Copy link
Copy Markdown
Contributor

No description provided.

@ivanbasov ivanbasov added Area-Compilers Test Test failures in roslyn-CI Interactive-EnC labels Jul 5, 2019
@ivanbasov ivanbasov requested review from 333fred, gafter and tmat July 5, 2019 23:42
@ivanbasov ivanbasov requested a review from a team as a code owner July 5, 2019 23:42
@ivanbasov ivanbasov requested a review from jcouv July 5, 2019 23:42
Copy link
Copy Markdown
Member

@gafter gafter left a comment

Choose a reason for hiding this comment

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

Please fix the problem exposed by the failing test.

@jcouv jcouv added this to the 16.3 milestone Jul 8, 2019
Copy link
Copy Markdown
Member

@gafter gafter left a comment

Choose a reason for hiding this comment

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

:shipit:

@gafter gafter merged commit d4a4944 into dotnet:master Jul 11, 2019
@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

@ivanbasov This appears to have broken builds completely. All my builds are failing with:

F:\workspace\_work\1\s\src\Compilers\CSharp\Test\Emit\Emit\EditAndContinue\EditAndContinueTests.cs(9856,85): error CS0117: 'ConditionalSkipReason' does not contain a definition for 'EditAndContinueRequiresDesktop' [F:\workspace\_work\1\s\src\Compilers\CSharp\Test\Emit\Microsoft.CodeAnalysis.CSharp.Emit.UnitTests.csproj]
F:\workspace\_work\1\s\src\Compilers\CSharp\Test\Emit\Emit\EditAndContinue\EditAndContinueTests.cs(9856,85): error CS0117: 'ConditionalSkipReason' does not contain a definition for 'EditAndContinueRequiresDesktop' [F:\workspace\_work\1\s\src\Compilers\CSharp\Test\Emit\Microsoft.CodeAnalysis.CSharp.Emit.UnitTests.csproj]
    0 Warning(s)
    2 Error(s)

Tagging @jasonmalinowski @dotnet/roslyn-infrastructure

See https://dev.azure.com/dnceng/public/_build/results?buildId=261072&view=logs for example.

@tmat
Copy link
Copy Markdown
Member

tmat commented Jul 11, 2019

Likely bad merge. Fixing.

@tmat tmat mentioned this pull request Jul 11, 2019
333fred added a commit to 333fred/roslyn that referenced this pull request Jul 11, 2019
* dotnet/master:
  Fix master build (dotnet#37163)
  Add error for multiple null suppressions (dotnet#37139)
  Check for uninitialized non-nullable field-like events (dotnet#37073)
  EnC emit tests for C# 8.0 patterns (dotnet#37025)
  Update dependencies from https://github.com/dotnet/arcade build 20190710.8
  Update dependencies from https://github.com/dotnet/arcade build 20190709.6
  [master] Update dependencies from dotnet/arcade (dotnet#37001)
  Add annotations to HostWorkspaceServices.cs and HostLanguageServices.cs
  Add nullable annotations to members in Solution.cs, SolutionInfo.cs, and SolutionState.cs
  Annotate Contract.cs
  Add nullable annotations and fixes to Workspace.cs
  Fix typo in documentation comment for Workspace.Dispose
  Check in copy of NullableAttributes.cs copied from coreclr
  Use switch expressions to simplify GetAccessibilityAndModifiers
@jasonmalinowski
Copy link
Copy Markdown
Member

@CyrusNajmabadi The build has been fixed up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants