Closed oesteban closed 1 year ago
No strong opinion on get_
or not.
Base: 85.31% // Head: 85.71% // Increases project coverage by +0.39%
:tada:
Coverage data is based on head (
2227ce4
) compared to base (3c1f2cb
). Patch coverage: 100.00% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
This patch allows users to access the automated querying of the layout with the
get_*
magic (andls_*
, for TF), e.g.:The existing
get_*
methods are not overshadowed:The original
get_*
queries are accepted:I'm hesitant about whether we should disallow
get_*
(as the meaning for TF is "fetch it" instead of getting from a table).WDYT?