Skip to content

Target .NET Core 3.0 in threading library - #548

Merged
Andrew Arnott (AArnott) merged 8 commits into
microsoft:masterfrom
AArnott:netcoreapp3.0
Nov 12, 2019
Merged

Target .NET Core 3.0 in threading library#548
Andrew Arnott (AArnott) merged 8 commits into
microsoft:masterfrom
AArnott:netcoreapp3.0

Conversation

@AArnott

@AArnott Andrew Arnott (AArnott) commented Nov 3, 2019

Copy link
Copy Markdown
Member

This gets us closer to supporting our DispatcherExtensions class on .NET Core 3.0, and immediately helps us get our nullable ref type annotations more correct.

Issues to resolve:

  • Warning MSB3277: Found conflicts between different versions of "System.IO.Pipes.AccessControl" that could not be resolved.
  • Add DispatcherExtensions to vs-threading's .NET Core 3.0 target (blocked by NETSDK1100 blocks building on Linux dotnet/sdk#3592)
  • Fix build breaks on Mac/Linux due to .NET SDK trying to be too clever.

This gets us closer to supporting our DispatcherExtensions class on .NET Core 3.0, and immediately helps us get our nullable ref type annotations more correct.
@sharwell

Copy link
Copy Markdown
Contributor

Looking forward to reviewing this one!

Comment thread src/Microsoft.VisualStudio.Threading.Tests/AsyncLazyTests.cs Outdated
Comment thread src/Microsoft.VisualStudio.Threading.Tests/AsyncSemaphoreTests.cs Outdated
Comment thread src/Microsoft.VisualStudio.Threading.Tests/AsyncSemaphoreTests.cs Outdated
Comment thread src/Microsoft.VisualStudio.Threading.Tests/AsyncSemaphoreTests.cs Outdated
Comment thread src/Microsoft.VisualStudio.Threading/AsyncAutoResetEvent.cs Outdated
Comment thread src/Microsoft.VisualStudio.Threading/AsyncReaderWriterResourceLock.cs Outdated
Comment thread src/Microsoft.VisualStudio.Threading/JoinableTaskFactory.cs Outdated
Comment thread src/Microsoft.VisualStudio.Threading/JoinableTask.cs
@AArnott
Andrew Arnott (AArnott) merged commit 8085d15 into microsoft:master Nov 12, 2019
@AArnott
Andrew Arnott (AArnott) deleted the netcoreapp3.0 branch November 12, 2019 03:29
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.

2 participants