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

Update compiler toolset to arcade version #36549

Merged
merged 1 commit into from
Jun 19, 2019

Conversation

agocke
Copy link
Member

@agocke agocke commented Jun 18, 2019

Fixes #36037

@agocke agocke marked this pull request as ready for review June 19, 2019 00:46
@agocke agocke requested a review from 333fred June 19, 2019 00:46
@agocke
Copy link
Member Author

agocke commented Jun 19, 2019

fyi @333fred it looks like arcade hasn't pushed an update to us in a month because a bad PR was open. I've closed the PR, but the fastest way to get this change in is to fix it directly.

@agocke agocke requested a review from a team June 19, 2019 00:47
@333fred
Copy link
Member

333fred commented Jun 19, 2019

Do we need to update the documentation for what version of the SDK to install as well?

@agocke
Copy link
Member Author

agocke commented Jun 19, 2019

SDK shouldn't be affected -- the package overrides the tasks/targets so it overrides whatever's in the SDK anyway

@agocke agocke merged commit 63e83ef into dotnet:master Jun 19, 2019
@agocke agocke deleted the use-arcade-toolset-version branch June 19, 2019 17:39
333fred added a commit to 333fred/roslyn that referenced this pull request Jun 20, 2019
* dotnet/master: (85 commits)
  Don't complete statement when typing semicolon inside comments in an argument list (dotnet#36521)
  Set focus to editor before finding text
  Add a bunch of nullability support to some code generation helpers
  Add 'annotations' and 'warnings' support to nullable directive (dotnet#36464)
  Fixed IDE services touching `notnull` constraint (dotnet#36508)
  Update compiler toolset to arcade version (dotnet#36549)
  Fix for 923157
  Do not include value types in NullableAttribute byte[] (dotnet#36519)
  Fix a deadlock in InvokeOnUIThread
  Apply a hang mitigating timeout to UI thread operations
  Move to a different lowering from for nullable value types to work around a bug in TransformCompoundAssignmentLHS. Addressed PR feedback.
  Fix stack overflow in requesting syntax directives (dotnet#36347)
  crash on ClassifyUpdate for EventFields (dotnet#35962)
  fixing bad merge with refactoring that was checked in later
  added basic completion statement telemetry
  Remove duplication in AbstractSymbolCompletionProvider.CreateItems
  Disable move type when the options service isn't present (dotnet#36334)
  Fix crash where type inference doing method inference needs to drop nullability
  Revert "Use IVsSolution to look up IVsHierarchy by project GUID (dotnet#35746)"
  Fix parsing bug in invalid using statements (dotnet#36428)
  ...
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.

Use source-build version of Microsoft.Net.Compilers.Toolset
2 participants