-
Notifications
You must be signed in to change notification settings - Fork 4.9k
API-diff between .NET 6.0 and .NET 7.0 #7976
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Windows Forms looks expected, though the diff is hard to parse.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK for System.Transactions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No direct changes for WPF. LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I have found 2 issues, PTAL at my comments.
release-notes/7.0/7.0.0/api-diff/Microsoft.NETCore.App/7.0.0_System.IO.md
Outdated
Show resolved
Hide resolved
release-notes/7.0/7.0.0/api-diff/Microsoft.NETCore.App/7.0.0_System.IO.md
Outdated
Show resolved
Hide resolved
System.Diagnostics.Activity, System.Diagnostics.Metrics, and System.ComponentModel LGTM. I assume it is intentional to list the nullability differences in the existing APIs too. Right? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RateLimiting looks good. Nullability in AspNetCore is off.
...tes/7.0/7.0.0/api-diff/Microsoft.AspNetCore.App/7.0.0_Microsoft.AspNetCore.Components.Web.md
Outdated
Show resolved
Hide resolved
...0/7.0.0/api-diff/Microsoft.AspNetCore.App/7.0.0_Microsoft.AspNetCore.Connections.Features.md
Outdated
Show resolved
Hide resolved
release-notes/7.0/7.0.0/api-diff/Microsoft.AspNetCore.App/7.0.0_Microsoft.AspNetCore.Http.md
Outdated
Show resolved
Hide resolved
release-notes/7.0/7.0.0/api-diff/Microsoft.AspNetCore.App/7.0.0_Microsoft.AspNetCore.Http.md
Outdated
Show resolved
Hide resolved
+ public static IResult Text(string content, string contentType = null, Encoding contentEncoding = null, int? statusCode = default(int?)); | ||
+ public static IResult UnprocessableEntity<TValue>(TValue error); | ||
- public static IResult ValidationProblem(IDictionary<string, string[]> errors, string? detail = null, string? instance = null, int? statusCode = default(int?), string? title = null, string? type = null, IDictionary<string, object?>? extensions = null); | ||
+ public static IResult ValidationProblem(IDictionary<string, string[]> errors, string detail = null, string instance = null, int? statusCode = default(int?), string title = null, string type = null, IDictionary<string, object?>? extensions = null); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More wrong nullability... I'm going to stop commenting on nullability inaccuracies otherwise I might be here all week
public interface IStatusCodeReExecuteFeature { | ||
+ Endpoint Endpoint { get; } | ||
- string? OriginalQueryString { get; set; } | ||
+ string OriginalQueryString { get; set; } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong nullability
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Microsoft.VisualBasic LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
System.Runtime.Serialization looks mostly good with just a couple suggestions.
release-notes/7.0/7.0.0/api-diff/Microsoft.NETCore.App/7.0.0_System.Runtime.Serialization.md
Outdated
Show resolved
Hide resolved
...7.0/7.0.0/api-diff/Microsoft.NETCore.App/7.0.0_System.Runtime.Serialization.DataContracts.md
Show resolved
Hide resolved
...7.0/7.0.0/api-diff/Microsoft.NETCore.App/7.0.0_System.Runtime.Serialization.DataContracts.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Brennan <brecon@microsoft.com> Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com> Co-authored-by: Steve Molloy <smolloy@microsoft.com>
...tes/7.0/7.0.0/api-diff/Microsoft.AspNetCore.App/7.0.0_Microsoft.AspNetCore.Components.Web.md
Show resolved
Hide resolved
…0_Microsoft.AspNetCore.Components.Web.md Co-authored-by: Brennan <brecon@microsoft.com>
ASP.NET - @dotnet/aspnet-api-review
WinForms - @merriemcgaw @RussKie
WPF - @singhashish-wpf and @pchaurasia14
Libraries - @jeffhandley @karelz @ericstj @stephentoub
Microsoft.VisualBasic: @jaredpar @cston @333fred
Microsoft.Win32: @dotnet/area-microsoft-win32
System.Buffers: @dotnet/area-system-buffers
System.CodeDom: @dotnet/area-system-codedom
System.Collections: @dotnet/area-system-collections
System.ComponentModel: @dotnet/area-system-componentmodel
System.ComponentModel.Composition: @dotnet/area-system-componentmodel-composition
System.Data: @ajcvickers @DavoudEshtehari @David-Engel @JRahnama
System.Diagnostics.Activity: @dotnet/area-system-diagnostics-activity
System.Diagnostics.CodeAnalysis: @AaronRobinsonMSFT
System.Diagnostics.EventLog: @dotnet/area-system-diagnostics-eventlog
System.Diagnostics.Metrics: @tommcdon @noahfalk
System.Diagnostics.PerformanceCounter: @dotnet/area-system-diagnostics-performancecounter
System.Diagnostics.Process: @dotnet/area-system-diagnostics-process
System.Formats.Tar: @dotnet/area-system-io
System.IO: @dotnet/area-system-io
System.IO.Compression: @dotnet/area-system-io-compression
System.Linq: @dotnet/area-system-linq
System.Net: @dotnet/ncl
System.Numerics: @dotnet/area-system-numerics
System.Reflection: @dotnet/area-system-reflection
System.Reflection.Emit: @dotnet/area-system-reflection-emit
System.Reflection.Metadata: @dotnet/area-system-reflection-metadata
System.Resources: @dotnet/area-system-resources
System.Runtime.CompilerServices: @dotnet/area-system-runtime-compilerservices
System.Runtime.InteropServices: @dotnet/interop-contrib @lewing @pavelsavara
System.Runtime.Intrinsics: @dotnet/area-system-runtime-intrinsics
System.Runtime.Serialization: @HongGit @StephenMolloy
System.Runtime.Versioning: @buyaa-n
System.Runtime: @dotnet/area-system-runtime
System.Security: @dotnet/area-system-security
System.Text: @dotnet/area-system-text-encoding
System.Text.Json: @dotnet/area-system-text-json
System.Text.RegularExpressions: @dotnet/area-system-text-regularexpressions
System.Threading: @mangod9 @kouvel
System.Timers: @dotnet/area-system-componentmodel
System.Transactions: @HongGit @roji
System.Xml: @dotnet/area-system-xml
System.Web: @Rick-Anderson
System: @dotnet/area-system-runtime