Closed khaeru closed 4 months ago
Attention: 8 lines
in your changes are missing coverage. Please review.
Comparison is base (
7a2de97
) 98.72% compared to head (13bf7ed
) 96.41%.
The codecov/patch check failure here is expected: there were added lines in data source tests that are not run on every PR commit, but only on main
on a schedule trigger. These should be run the first time that happens after the PR is merged.
Closes #162.
Note that this PR does not address this TODO: https://github.com/khaeru/sdmx/blob/7a2de97237860eac894399fd8c4cba9e09180234/sdmx/client.py#L279-L283 …as a consequence, some of the code in .source.estat.handle_references_param() is duplicative of existing code in Client._handle_get_kwargs(). The Source class will get a more extensive rework soon, in which this duplication can probably be removed.
PR checklist