Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(config): Default the entire config #768

Merged
merged 1 commit into from
Nov 25, 2023
Merged

Conversation

afnanenayet
Copy link
Owner

Make the config struct use defaults for the entire config instead of
just a few sections.

Make the config struct use defaults for the entire config instead of
just a few sections.
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4ea668b) 59.47% compared to head (b3c3b1e) 59.47%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #768   +/-   ##
=======================================
  Coverage   59.47%   59.47%           
=======================================
  Files          13       13           
  Lines        1451     1451           
=======================================
  Hits          863      863           
  Misses        588      588           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@afnanenayet afnanenayet merged commit 1da051a into main Nov 25, 2023
@afnanenayet afnanenayet deleted the afnan/config-defaults branch November 25, 2023 04:35
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants