Update info on rendering of C# data types to OpenAPI in .NET 10 #35222
Conversation
Co-authored-by: Rick Anderson <[email protected]>
|
@Rick-Anderson I pushed one new commit to update the information about how nullable properties appear in the OpenAPI doc, as this has also changed in .NET 10. |
d1f57a2 looks good. I'm fixing a couple build errors. |
| | int | [integer,string] | int32 | pattern `<digits>` | pattern: `"^-?(?:0|[1-9]\\d*)$"` | ||
| | long | [integer,string] | int64 | pattern `<digits>` | | ||
| | short | [integer,string] | int16 | pattern `<digits>` | | ||
| | byte | [integer,string] | uint8 | pattern `<digits>` | | ||
| | float | [number,string] | float | pattern `<digits with decimal >` | pattern: `"^-?(?:0|[1-9]\\d*)(?:\\.\\d+)?(?:[eE][+-]?\\d+)?$"` | ||
| | double | [number,string] | double | pattern `<digits with decimal >` | | ||
| | decimal | [number,string] | double | pattern `<digits with decimal >` | pattern: `"^-?(?:0|[1-9]\\d*)(?:\\.\\d+)?$"` |
There was a problem hiding this comment.
@mikekistler
Took me awhile to figure that build error. See the | in the pattern? Even commented out, the parser picks up the column separator before stripping comment - error
even with ``` back ticks, same column error. Very strange
The other build error took a bit too, you had :: moniker-end, only two :, and 3 required. I've never made that mistake - every, because I'm dyslexic and can't type. I copy/paste.
There was a problem hiding this comment.
Oh @Rick-Anderson ... so sorry! I think I force-pushed and erased some of your changes by accident. I should have been more careful. I will try to recover.
There was a problem hiding this comment.
Okay ... I think I restored your version. I will be more careful from now on.
There was a problem hiding this comment.
Oh @Rick-Anderson ... so sorry! I think I force-pushed and erased some of your changes by accident. I should have been more careful. I will try to recover.
I should have made clear I pushed some commits, sorry. I thought you were finished.
mikekistler
force-pushed
the
mdk/dotnet10-openapi-metadata
branch
2 times, most recently
from
90c3421 to
aaa5fe4
Compare
|
@mikekistler merge when you're ready |
This PR updates the information on how ASP.NET Core renders C# data types into OpenAPI schema in .NET 10.
This is a companion PR to #35214.
Internal previews