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/dev17.6] Update dependencies from dotnet/arcade #73864

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jun 5, 2024

This pull request updates the following dependencies

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

  • Subscription: 803de9f2-7aae-49ed-19a8-08dc852fa6db
  • Build: 20240520.4
  • Date Produced: May 20, 2024 11:44:41 PM UTC
  • Commit: f2b2071632d5d4c46d0f904f2b0d917b1752551b
  • Branch: refs/heads/release/8.0
  • Updates to .NET SDKs:
    • Updates sdk.version to 8.0.100-preview.7.23376.3
    • Updates tools.dotnet to 8.0.100-preview.7.23376.3

…520.4

Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Helix.Sdk
 From Version 8.0.0-beta.23168.1 -> To Version 8.0.0-beta.24270.4
@dotnet-maestro dotnet-maestro bot requested a review from a team as a code owner June 5, 2024 22:33
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead labels Jun 5, 2024
@JoeRobich JoeRobich requested a review from a team as a code owner June 5, 2024 23:17
@JoeRobich JoeRobich requested a review from a team as a code owner June 5, 2024 23:22
@@ -32,7 +32,6 @@
<PackageReference Include="Nerdbank.Streams" Version="$(NerdbankStreamsVersion)" />
<PackageReference Include="System.Threading.Tasks.Dataflow" Version="$(SystemThreadingTasksDataflowVersion)" />
<PackageReference Include="System.Buffers" Version="$(SystemBuffersVersion)" />
<PackageReference Include="xunit.assert" Version="$(xunitassertVersion)" />
Copy link
Member

Choose a reason for hiding this comment

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

Backport removing xunit.assert from #68573

@@ -61,7 +61,6 @@
<PackageReference Include="Microsoft.VisualStudio.Imaging.Interop.14.0.DesignTime" Version="$(MicrosoftVisualStudioImagingInterop140DesignTimeVersion)" />
<PackageReference Include="NuGet.SolutionRestoreManager.Interop" Version="$(NuGetSolutionRestoreManagerInteropVersion)" />
<PackageReference Include="UIAComWrapper" Version="$(UIAComWrapperVersion)" />
<PackageReference Include="xunit.assert" Version="$(xunitassertVersion)" />
Copy link
Member

Choose a reason for hiding this comment

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

Backport removing xunit.assert from #68573

@@ -119,7 +119,9 @@ public void WriteTo(ObjectWriter writer)
public void WriteTo(Span<byte> span)
{
Contract.ThrowIfFalse(span.Length >= HashSize);
#pragma warning disable CS9191 // The 'ref' modifier for an argument corresponding to 'in' parameter is equivalent to 'in'. Consider using 'in' instead.
Copy link
Member

Choose a reason for hiding this comment

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

Backport pragma disable from #70248

global.json Outdated Show resolved Hide resolved
@@ -7,7 +7,6 @@
</PropertyGroup>

<ItemGroup>
<PackageReference Include="xunit.analyzers" Version="$(xunitanalyzersVersion)" IsImplicitlyDefined="true" PrivateAssets="all" />
Copy link
Member

Choose a reason for hiding this comment

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

Backport XUnit.targets change from #68573

global.json Outdated Show resolved Hide resolved
@JoeRobich
Copy link
Member

/azp run

JoeRobich and others added 3 commits June 10, 2024 15:59
…se/dev17.6-f1969d28-bf05-4aad-988c-dc5bd3651ad8
The EndToEnd OverflowOnFluentCall test is fixed in #67913, but would be a significant change to backport.
<>F{00000001}`2[System.Int32,(fnptr)[]]
<>A{00000001}`1[(fnptr)[]]
");
CompileAndVerify(comp, expectedOutput: ExecutionConditionUtil.IsCoreClr
Copy link
Member

Choose a reason for hiding this comment

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

Backport test fixes from #68212

@@ -2479,7 +2479,7 @@ static void Main()
Console.WriteLine(t.ToString());
}
}
", expectedOutput: "System.IntPtr");
", expectedOutput: ExecutionConditionUtil.IsCoreClr ? "System.Void()" : "System.IntPtr");
Copy link
Member

Choose a reason for hiding this comment

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

Backport test fixes from #68212

@@ -6,6 +6,10 @@
https://devdiv.pkgs.visualstudio.com/_packaging/Engineering/nuget/v3/index.json;
https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-public/nuget/v3/index.json;
</RestoreSources>
<RestoreSources>
Copy link
Member

Choose a reason for hiding this comment

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

Backport RestoreSource from #73532

Copy link
Member

@jjonescz jjonescz Jun 12, 2024

Choose a reason for hiding this comment

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

It doesn't look like there's any meaningful change - only whitespace removed. I'm fine with that, but wondering if that is intentional?

EDIT: I guess that's because the target branch changed in #73950.

Copy link
Member

Choose a reason for hiding this comment

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

That's right. We both had the same change but differing whitespace.

@JoeRobich JoeRobich requested review from jjonescz and dibarbet June 12, 2024 05:03
@JoeRobich
Copy link
Member

JoeRobich commented Jun 12, 2024

With Arcade updated we could move the 17.6 official build to 1ES PT which would get us past the current build blockers of unnecessary build parameters and missing variables. (See run)

@@ -6,6 +6,10 @@
https://devdiv.pkgs.visualstudio.com/_packaging/Engineering/nuget/v3/index.json;
https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-public/nuget/v3/index.json;
</RestoreSources>
<RestoreSources>
Copy link
Member

@jjonescz jjonescz Jun 12, 2024

Choose a reason for hiding this comment

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

It doesn't look like there's any meaningful change - only whitespace removed. I'm fine with that, but wondering if that is intentional?

EDIT: I guess that's because the target branch changed in #73950.

@JoeRobich JoeRobich merged commit e58065d into release/dev17.6 Jun 12, 2024
23 of 28 checks passed
@dotnet-maestro dotnet-maestro bot deleted the darc-release/dev17.6-f1969d28-bf05-4aad-988c-dc5bd3651ad8 branch June 12, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants