-
Notifications
You must be signed in to change notification settings - Fork 4.9k
API Diff between .NET 10.0 Preview 5 and Preview 6 #9976
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
Conversation
We may think in improving the tool formatting as I am seeing it is mis-indent some lines especially with const like |
...0/preview/preview6/api-diff/Microsoft.NETCore.App/10.0-preview6_System.Runtime.Intrinsics.md
Show resolved
Hide resolved
...e-notes/10.0/preview/preview6/api-diff/Microsoft.NETCore.App/10.0-preview6_System.Runtime.md
Show resolved
Hide resolved
@tarekgh |
Thanks @OwnageIsMagic for the catch. @ericstj It looks like we need to fix this issue in the tool. The enum isn’t new, yet the tool is generating it as a class. It seems the entire System.Formats.Cbor code being generated shouldn't be there at all. + namespace System.Formats.Cbor
+ {
+ public sealed class CborConformanceMode
+ {
+ public const System.Formats.Cbor.CborConformanceMode Canonical = 2;
+ public const System.Formats.Cbor.CborConformanceMode Ctap2Canonical = 3;
+ public const System.Formats.Cbor.CborConformanceMode Lax = 0;
+ public const System.Formats.Cbor.CborConformanceMode Strict = 1;
+ public int value__;
+ }
+ } |
That's a bizarre bug - the enum hasn't been touched for some time. What changed was the I think I see where the bug is, I'll put up a PR. |
I opened #9988 - thank you for noticing this. |
Repo area owners: