Closed alperkaya closed 3 weeks ago
Thanks for the PR! By convention, we like prefix such functions with _inplace
. Could you rename it to deserialize_document_store_in_init_params_inplace
instead (and update the docstring to mention the inplace action)?
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Totals | |
---|---|
Change from base Build 10581746861: | -0.001% |
Covered Lines: | 7004 |
Relevant Lines: | 7765 |
Related Issues
Proposed Changes:
How did you test it?
CI
Notes for the reviewer
Checklist
fix:
,feat:
,build:
,chore:
,ci:
,docs:
,style:
,refactor:
,perf:
,test:
.