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/installer #93921

Merged
merged 2 commits into from
Oct 24, 2023

Conversation

dotnet-maestro[bot]
Copy link
Contributor

This pull request updates the following dependencies

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

  • Subscription: 2a4e5111-6978-401f-68a5-08dba3b24dbe
  • Build: 20231024.2
  • Date Produced: October 24, 2023 9:31:54 AM UTC
  • Commit: ba5b9001a0a61ddff041886745b0b395623a9227
  • Branch: refs/heads/main

…231024.2

Microsoft.Dotnet.Sdk.Internal
 From Version 9.0.100-alpha.1.23522.3 -> To Version 9.0.100-alpha.1.23524.2
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-codeflow for labeling automated codeflow label Oct 24, 2023
@radical
Copy link
Member

radical commented Oct 24, 2023

LazyLoadingTest.cs(15,9): Trim analysis error IL2026: LazyLoadingTest.Run(): Using member 'System.Text.Json.JsonSerializer.Serialize<TValue>(TValue, JsonSerializerOptions)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. JSON serialization and deserialization might require types that cannot be statically analyzed. Use the overload that takes a JsonTypeInfo or JsonSerializerContext, or make sure all of the required types are preserved.

@maraf New analyzer breaking the TestAppScenarios* tests.

update: log

@radical radical self-assigned this Oct 24, 2023
`[] /root/helix/work/workitem/e/wbt/AppSettingsTests_haqswfqc_rzt/LazyLoadingTest.cs(15,9): Trim analysis error IL2026: LazyLoadingTest.Run(): Using member 'System.Text.Json.JsonSerializer.Serialize<TValue>(TValue, JsonSerializerOptions)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. JSON serialization and deserialization might require types that cannot be statically analyzed. Use the overload that takes a JsonTypeInfo or JsonSerializerContext, or make sure all of the required types are preserved. [/root/helix/work/workitem/e/wbt/AppSettingsTests_haqswfqc_rzt/WasmBasicTestApp.csproj]`
@radical
Copy link
Member

radical commented Oct 24, 2023

Failing tests are #93522 .

@maraf please feel free to change LazyLoadingTests.cs if you want to remove the warning suppression.

@radical radical merged commit 08903c0 into main Oct 24, 2023
183 of 186 checks passed
@radical radical deleted the darc-main-760105b2-2776-43d2-ad4f-a45fc59f5f30 branch October 24, 2023 23:22
liveans pushed a commit to liveans/dotnet-runtime that referenced this pull request Nov 9, 2023
* Update dependencies from https://github.com/dotnet/installer build 20231024.2

Microsoft.Dotnet.Sdk.Internal
 From Version 9.0.100-alpha.1.23522.3 -> To Version 9.0.100-alpha.1.23524.2

* Suppress trim warnings

`[] /root/helix/work/workitem/e/wbt/AppSettingsTests_haqswfqc_rzt/LazyLoadingTest.cs(15,9): Trim analysis error IL2026: LazyLoadingTest.Run(): Using member 'System.Text.Json.JsonSerializer.Serialize<TValue>(TValue, JsonSerializerOptions)' which has 'RequiresUnreferencedCodeAttribute' can break functionality when trimming application code. JSON serialization and deserialization might require types that cannot be statically analyzed. Use the overload that takes a JsonTypeInfo or JsonSerializerContext, or make sure all of the required types are preserved. [/root/helix/work/workitem/e/wbt/AppSettingsTests_haqswfqc_rzt/WasmBasicTestApp.csproj]`

---------

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Ankit Jain <radical@gmail.com>
@ghost ghost locked as resolved and limited conversation to collaborators Nov 24, 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant