forked from RicoSuter/NJsonSchema
-
Notifications
You must be signed in to change notification settings - Fork 0
Sync to original #1
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
Open
statler
wants to merge
1,263
commits into
statler:master
Choose a base branch
from
RicoSuter:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
….Json (#1492) * fix: system.text.json enum naming policy * Update SystemTextJsonUtilities.cs Co-authored-by: Rico Suter <[email protected]>
* Add support for TypeScript 4.3 override keyword * Create verify tests for different output styles and TS versions
* Update CSharpPropertyNameGenerator.cs Added ampersand sign (&) replacement in name generator. * Update CSharpPropertyNameGenerator.cs Co-authored-by: Rico Suter <[email protected]>
* Generate STJ JsonIgnoreAttributes for model properties * Update PropertyModel.cs Co-authored-by: Rico Suter <[email protected]>
* Update JsonInheritanceConverter.liquid The InheritanceConverters needs to be made public so downstream NSwag Generator can get the DiscriminatorName (i.e. wrapping Client classes in another generated client class) Error below: 6> ---> Microsoft.CSharp.RuntimeBinder.RuntimeBinderException: 'System.Text.Json.Serialization.JsonConverter<BaseClass>' does not contain a definition for 'DiscriminatorName' * updated missing namepace for JsonException * fix unit tests Co-authored-by: Mike Mao <[email protected]>
Co-authored-by: abulat <[email protected]>
* #250 Allow case insensitive property validation * Handle ignore casing on required properties not in schema * Rename 'JsonSchemaValidatorSettings' to 'JsonSchemaValidatorOptions' * Allow using custom StringComparer for property validation
…lity issues. (#1520) * Re-add old JsonSchemaValidator constructor to fix backwards compatability issues. * Update JsonSchemaValidator.cs Co-authored-by: Rico Suter <[email protected]>
This allows to implement different logic e.g. treat JsonSchema.OneOf to have multiple (non nullable) items instead of zero or one.
* Fix output of JsonIgnore attributes * Recognize Guid as value type * Rename variable * Use JsonFormatStrings for ValueTypeFormats * Easier fix * Remove unused field
* Add support for flag enums with more than 32 values * Update EnumTests.cs Co-authored-by: Christoph Genneper <[email protected]> Co-authored-by: Rico Suter <[email protected]>
#1835) Co-authored-by: Georgy Losenkov <[email protected]>
* Allow disabling sorting of constructor parameters * Base class parameters should come before derived class parameters --------- Co-authored-by: Marko Lahma <[email protected]>
* make regexes lazy * cleanup
- Add test cases for enum descriptions and backward compatibility - Add test case for enum descriptions with Description attributes - Add backward compatibility tests for x-enum-names, x-enum-varnames, and x-enum-descriptions --------- Co-authored-by: Marko Lahma <[email protected]>
* use compiled regex in CSharpDocs, check if replacement needed * proper constant stack allocation size in ConversionUtilities * better replace behavior in JsonReferenceVisitorBase
* Improve enum description handling and make reading tolerant to invalid data * don't output multiple versions, just support reading multiple
) Unlike JsonStringEnumConverter, the generic JsonStringEnumConverter<T> is AOT compatible. This prevents the [SYSLIB1034](https://learn.microsoft.com/en-us/dotnet/fundamentals/syslib-diagnostics/syslib1034) warning to occur when System.Text.Json source generation is used.
System.Text.Json ignores the `System.Runtime.Serialization.EnumMember` attribute. It needs the [`System.Text.Json.Serialization.JsonStringEnumMemberName` attribute](https://learn.microsoft.com/en-us/dotnet/standard/serialization/system-text-json/customize-properties#custom-enum-member-names) to customize the names of individual enum members.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.