-
Notifications
You must be signed in to change notification settings - Fork 450
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
Remove pydantic from config #1621
Remove pydantic from config #1621
Conversation
superduperdb/base/logger.py
Outdated
@@ -14,6 +14,9 @@ | |||
__all__ = ('Logging',) | |||
|
|||
|
|||
breakpoint() |
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.
Not intended probably?
class BaseConfigJSONable(JSONable): | ||
@dc.dataclass | ||
class BaseConfigJSONable: | ||
_lock: t.ClassVar[bool] = False |
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.
What does this do?
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.
This is for locking the config after initialisation
cac4f3d
to
f330ef9
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1621 +/- ##
==========================================
+ Coverage 80.33% 80.64% +0.30%
==========================================
Files 95 111 +16
Lines 6602 7899 +1297
==========================================
+ Hits 5304 6370 +1066
- Misses 1298 1529 +231 ☔ View full report in Codecov by Sentry. |
Description
Fix #1609
Related Issues
Checklist
make unit-testing
andmake integration-testing
successfully?Additional Notes or Comments