Skip to content
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

Add support for default values #212

Merged
merged 8 commits into from
May 29, 2023
Merged

Add support for default values #212

merged 8 commits into from
May 29, 2023

Conversation

tlsa
Copy link
Owner

@tlsa tlsa commented May 28, 2023

This allows the schema to specify a value to use when an optional mapping field is missing.

Fixes #96.

@tlsa tlsa added this to the Version 2 milestone May 28, 2023
@codecov
Copy link

codecov bot commented May 28, 2023

Codecov Report

Merging #212 (95afbb8) into main (9b6409c) will increase coverage by 0.01%.
The diff coverage is 97.39%.

@@            Coverage Diff             @@
##             main     #212      +/-   ##
==========================================
+ Coverage   97.11%   97.12%   +0.01%     
==========================================
  Files          10       10              
  Lines        1905     2017     +112     
  Branches      407      424      +17     
==========================================
+ Hits         1850     1959     +109     
- Misses         26       28       +2     
- Partials       29       30       +1     
Flag Coverage Δ
unittests 97.12% <97.39%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/load.c 98.51% <97.39%> (-0.15%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tlsa tlsa force-pushed the tlsa/default-values branch 3 times, most recently from dbfb5cb to 7bed63b Compare May 28, 2023 18:57
@tlsa tlsa force-pushed the tlsa/default-values branch from 7bed63b to 95afbb8 Compare May 29, 2023 12:57
@tlsa tlsa merged commit c637a24 into main May 29, 2023
@tlsa tlsa deleted the tlsa/default-values branch May 29, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add default value to fields
1 participant