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

[release/8.0] Update dependencies from dotnet/roslyn #91913

Merged

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Sep 12, 2023

This pull request updates the following dependencies

From https://github.com/dotnet/roslyn

  • Subscription: ac2d12d7-7c98-40fe-ec67-08db9e4ff4ff
  • Build: 20230912.10
  • Date Produced: September 13, 2023 7:28:09 AM UTC
  • Commit: 1b7a6f807cb8ce709048debae6b771f4705a697a
  • Branch: refs/heads/main

…911.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23461.1
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-codeflow for labeling automated codeflow label Sep 12, 2023
…911.4

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23461.4
…912.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.1
@carlossanlop carlossanlop added the Servicing-approved Approved for servicing release label Sep 12, 2023
carlossanlop and others added 7 commits September 12, 2023 09:00
…912.2

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.2
…912.3

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.3
…912.4

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.4
…912.6

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.6
…912.7

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.7
…912.8

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.8
…912.9

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.9
…912.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.8.0-3.23459.2 -> To Version 4.8.0-3.23462.10
Copy link
Member

@ericstj ericstj left a comment

Choose a reason for hiding this comment

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

Do not merge this PR until we've addressed the failing reflection tests.

System.Tests.Types.GenericInterface1Tests.Attributes_Get_ReturnsExpected

Error message
Assert.Equal() Failure
Expected: Public | ClassSemanticsMask | Abstract
Actual:   Public | ClassSemanticsMask | Abstract | BeforeFieldInit

Stack trace
   at System.Tests.Types.TypePropertyTestBase.Attributes_Get_ReturnsExpected() + 0x7c
   at System.Runtime.Tests!<BaseAddress>+0x1614bf0
   at System.Reflection.DynamicInvokeInfo.Invoke(Object, IntPtr, Object[], BinderBundle, Boolean) + 0xcc

@ghost ghost added the needs-author-action An issue or pull request that requires more info or actions from the author. label Sep 13, 2023
@ghost ghost removed the needs-author-action An issue or pull request that requires more info or actions from the author. label Sep 13, 2023
Copy link
Member

@ericstj ericstj left a comment

Choose a reason for hiding this comment

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

I think this should fix the failing reflection tests @steveharter @AlekseyTs

@carlossanlop
Copy link
Member

I merged the revert PR. I am now restarting the CI of the PRs that caught that failure so that they get better results.

@carlossanlop
Copy link
Member

CI is good again. Merging. cc @ericstj

@carlossanlop carlossanlop merged commit dbcde1d into release/8.0 Sep 14, 2023
187 of 188 checks passed
@carlossanlop carlossanlop deleted the darc-release/8.0-71360d31-7c01-4d25-a66c-aafc442afc94 branch September 14, 2023 19:21
@radical radical mentioned this pull request Sep 26, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Oct 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-codeflow for labeling automated codeflow Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants