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

Disable WinRT tests on Linux #31026

Merged
merged 1 commit into from
Nov 7, 2018

Conversation

agocke
Copy link
Member

@agocke agocke commented Nov 7, 2018

They are occasionally crashing in the runtime on Linux e.g., https://dnceng.visualstudio.com/public/_build/results?buildId=40756&view=logs

@agocke agocke requested a review from a team as a code owner November 7, 2018 20:45
@agocke agocke merged commit 60947cb into dotnet:master Nov 7, 2018
@agocke agocke deleted the disable-winrt-tests-on-linux branch November 7, 2018 23:08
wachulski added a commit to wachulski/roslyn that referenced this pull request Nov 12, 2018
…out-if-error-message

* origin/master: (174 commits)
  Add Compilers filter for Roslyn (dotnet#30880)
  Remove NonNullTypes context and other unnecessary information stored in TypeSymbolWithAnnotations. (dotnet#30913)
  Update BoundCall method based on receiver nullability (dotnet#31000)
  Make nullabiulity inference associative and commutative (dotnet#30990)
  Async-streams: minimal test for IOperation and CFG. Improve diagnostics (dotnet#30363)
  Add src.
  Add test.
  Remove dead code
  Update the build status table
  Script for generating our build status tables
  Add parsing tests to compiler benchmarks (dotnet#31033)
  Fix Edit and Continue in CPS projects
  Add comment
  Sorting.
  Save work
  Address PR feedback
  only produce optprof data on an official build
  Add bunch of exclusions to dead code analysis for special methods
  Disable WinRT tests on Linux (dotnet#31026)
  Change prerelease version to beta2 (dotnet#31017)
  ...

# Conflicts:
#	src/Compilers/CSharp/Portable/CSharpResources.resx
#	src/Compilers/CSharp/Portable/Errors/ErrorCode.cs
@tmat
Copy link
Member

tmat commented Nov 24, 2018

@agocke Is there an issue tracking enabling these?

@agocke
Copy link
Member Author

agocke commented Nov 27, 2018

The long-term plan I made with @jaredpar... years ago at this point, was that there's no need for us to run WinRT tests on anything but Windows Desktop and that that assembly could eventually morph into the Windows-Desktop-Only assembly that contains most of the by-design-doesn't-run-xplat tests.

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