-
Notifications
You must be signed in to change notification settings - Fork 82
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
feat: 🎸 ensure immutability of the configs #790
Conversation
Codecov ReportBase: 90.72% // Head: 90.72% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## main #790 +/- ##
=======================================
Coverage 90.72% 90.72%
=======================================
Files 105 105
Lines 5259 5259
=======================================
Hits 4771 4771
Misses 488 488
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
The documentation is not available anymore as the PR was closed or merged. |
if we decide to allow changing config parameters later, it will need to be explicit.
0ff381a
to
3f4af2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! It has more sense for configs
if we decide to allow changing config parameters later, it will need to be explicit.
Depends on: #784