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.

Sign up for GitHub

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

Jump to bottom

'require' in jsonschema tags will always override ',omitempty' #90

Merged

samlown merged 4 commits into main from require-override

Sep 6, 2023

Contributor

samlown commented Sep 6, 2023 •

edited

If the jsonschema:"require" tag is present, it will now always override the previously determined required status, and not just when the RequiredFromJSONSchemaTags is true.
Includes updated linter requirements (its been a while!)

samlown added 4 commits

September 6, 2023 14:43


          'require' in jsonschema tags will always override ',omitempty'

4a76377


          Updating test fixtures

55971c8


          Updating linter config

16ff337


          Fixing linting strategy

ed817e0

samlown mentioned this pull request

Add Reflector.Requirer to support custom struct field requiring #88

Open

samlown merged commit e63dc34 into main

3 checks passed

samlown deleted the require-override branch

September 6, 2023 15:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment