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

[main] Update dependencies from dotnet/roslyn #81164

Merged
merged 36 commits into from
Feb 21, 2023

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jan 25, 2023

This pull request updates the following dependencies

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

  • Subscription: 0af788c4-77ba-425c-b8c9-08da91234a73
  • Build: 20230220.1
  • Date Produced: February 20, 2023 10:43:44 AM UTC
  • Commit: 4190056e0126f206c64439275fbf43a54dd31067
  • Branch: refs/heads/main

…124.5

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-1.23074.5
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-codeflow for labeling automated codeflow label Jan 25, 2023
@lewing
Copy link
Member

lewing commented Jan 25, 2023

src/libraries/System.Runtime.InteropServices/tests/LibraryImportGenerator.UnitTests/AddDisableRuntimeMarshallingAttributeFixerTests.cs(145,22): error RS1036: (NETCORE_ENGINEERING_TELEMETRY=Build) 'LibraryImportGenerator.UnitTests.AddDisableRuntimeMarshallingAttributeFixerTests.MockAnalyzer': A project containing analyzers or source generators should specify the property '<EnforceExtendedAnalyzerRules>true</EnforceExtendedAnalyzerRules>' with the new roslyn

@ghost
Copy link

ghost commented Jan 25, 2023

Tagging subscribers to this area: @dotnet/interop-contrib
See info in area-owners.md if you want to be subscribed.

Issue Details

This pull request updates the following dependencies

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

  • Subscription: 0af788c4-77ba-425c-b8c9-08da91234a73
  • Build: 20230124.5
  • Date Produced: January 25, 2023 4:51:37 AM UTC
  • Commit: 234e15d0b7b74ad1119612ed214939928b5b4aea
  • Branch: refs/heads/main
Author: dotnet-maestro[bot]
Assignees: -
Labels:

area-System.Runtime.InteropServices, area-codeflow

Milestone: -

@jkoritzinsky
Copy link
Member

We can disable that diagnostic for that assembly. The analayzer there isn't shipping and is only to work around some limitations in the unit testing system.

dotnet-maestro bot and others added 2 commits January 26, 2023 13:48
…126.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-1.23076.1
@lewing lewing added area-Tools-ILLink .NET linker development as well as trimming analyzers and removed area-System.Runtime.InteropServices labels Jan 26, 2023
@ghost
Copy link

ghost commented Jan 26, 2023

Tagging subscribers to this area: @agocke, @sbomer, @vitek-karas
See info in area-owners.md if you want to be subscribed.

Issue Details

This pull request updates the following dependencies

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

  • Subscription: 0af788c4-77ba-425c-b8c9-08da91234a73
  • Build: 20230126.1
  • Date Produced: January 26, 2023 10:00:14 AM UTC
  • Commit: fe8f333ce7eac33aec8dd87fd1fc54d23e65b141
  • Branch: refs/heads/main
Author: dotnet-maestro[bot]
Assignees: -
Labels:

area-codeflow, area-Tools-ILLink

Milestone: -

@lewing
Copy link
Member

lewing commented Jan 26, 2023

src/tools/illink/src/ILLink.CodeFix/UnconditionalSuppressMessageCodeFixProvider.cs(47,58): error RS1035: (NETCORE_ENGINEERING_TELEMETRY=Build) The symbol 'CultureInfo.CurrentUICulture' is banned for use by analyzers: Analyzers should use the locale given by the compiler command line arguments, not the CurrentUICulture I'm not clear if this one is genuine or should be suppressed. cc @vitek-karas @sbomer

@jkoritzinsky
Copy link
Member

I asked @RikkiGibson how best to fix that diagnostic as I can't figure out the right API to use to get to the command line arguments. In the meantime, it's probably okay to suppress, just not the best. That one I'd suppress at the use-site.

…127.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-1.23077.1
…127.8

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-1.23077.8
…128.5

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-1.23078.5
…129.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-1.23079.1
…131.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23081.1
…131.23

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23081.23
@EgorBo
Copy link
Member

EgorBo commented Feb 1, 2023

So this is currently blocked on Roslyn folks to address dotnet/roslyn#66566 or we can ignore the warning untill it's fixed and land this?

…201.22

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23101.22
@jkoritzinsky
Copy link
Member

@EgorBo yes, that is correct.

dotnet-maestro bot and others added 4 commits February 3, 2023 13:53
…202.6

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23102.6
…203.10

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23103.10
…204.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23104.1
…214.7

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23114.7
@ghost
Copy link

ghost commented Feb 15, 2023

Tagging subscribers to this area: @dotnet/area-system-text-json, @gregsdennis
See info in area-owners.md if you want to be subscribed.

Issue Details

This pull request updates the following dependencies

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

  • Subscription: 0af788c4-77ba-425c-b8c9-08da91234a73
  • Build: 20230214.7
  • Date Produced: February 15, 2023 3:14:48 AM UTC
  • Commit: 12105175705e16f905dd1ec1fe8f7cff07ab8ba0
  • Branch: refs/heads/main
Author: dotnet-maestro[bot]
Assignees: -
Labels:

area-System.Text.Json, area-codeflow

Milestone: -

@lewing
Copy link
Member

lewing commented Feb 15, 2023

Has someone opened a roslyn issue for this?

@lewing
Copy link
Member

lewing commented Feb 15, 2023

dotnet/roslyn#66900

dotnet-maestro bot and others added 3 commits February 16, 2023 13:43
…215.9

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23115.9
…216.4

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23116.4
…217.4

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23117.4
…218.2

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23118.2
@lewing lewing added the blocked Issue/PR is blocked on something - see comments label Feb 20, 2023
@marek-safar marek-safar force-pushed the darc-main-5410199c-3511-41c9-8178-d0c10ae042d9 branch from 01e53a2 to 6283b2e Compare February 20, 2023 09:49
@marek-safar marek-safar force-pushed the darc-main-5410199c-3511-41c9-8178-d0c10ae042d9 branch from 6283b2e to 9c6c227 Compare February 20, 2023 10:50
dotnet-maestro bot and others added 2 commits February 20, 2023 13:30
…220.1

Microsoft.CodeAnalysis , Microsoft.CodeAnalysis.CSharp , Microsoft.Net.Compilers.Toolset
 From Version 4.6.0-1.23073.4 -> To Version 4.6.0-2.23120.1
@marek-safar marek-safar merged commit bbbc73e into main Feb 21, 2023
@marek-safar marek-safar deleted the darc-main-5410199c-3511-41c9-8178-d0c10ae042d9 branch February 21, 2023 08:13
@EgorBo
Copy link
Member

EgorBo commented Feb 21, 2023

🎉

hoyosjs added a commit that referenced this pull request Feb 22, 2023
hoyosjs added a commit that referenced this pull request Feb 22, 2023
Reverts version bump component of #81164

This PR caused heavy managed build regressions hitting all PR builds. See #82458 and #76454
jaredpar pushed a commit to jaredpar/runtime that referenced this pull request Feb 28, 2023
Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Jeremy Koritzinsky <[email protected]>
Co-authored-by: vitek-karas <[email protected]>
Co-authored-by: pavelsavara <[email protected]>
Co-authored-by: Marek Safar <[email protected]>
@ghost ghost locked as resolved and limited conversation to collaborators Mar 23, 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 area-System.Text.Json blocked Issue/PR is blocked on something - see comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants