Staging instance, all changes can be removed at any time

Skip to content

scanner.model: Fix Tree.toDict to be side-effect free

Tenma requested to merge generated-differential-D4186-source into generated-differential-D4186-target

Remove reliance on default arg child_nodes which is a dict.

It is unused in client code, breaks tests, and is not needed to build the dict representation of the tree.

This also refines types on related impacted methods which helps reasoning about them.

Related to !74 (closed)

Migrated from D4186 (view on Phabricator)

Merge request reports