Closed ianbrunjes closed 3 years ago
LGTM. just fixed the test suite issues on the old releases, should be good now :crossed_fingers:
Hmmm, looks like the checks failed again on those missing packages after retrigger from your PR.
ah, but those are just the checks on the branch, which doesn't have the changes. looks like we'll be fine after merge
Tests and check both pass locally. Since the changes to get/set attributes modify existing structures, no new tests were added (though certainly can be).