InnocentBug commented 8 months ago

Description

Add iterator capability for UUID nodes.

Changes

UUID attributes are strictly strings now.

Let's add some test to ensure iteration over nodes is dfs, and we need to tests this with a graph that contains cycles.

[ x] My name is on the list of contributors (CONTRIBUTORS.md) in the pull request source branch.
[ x] I have updated the documentation to reflect my changes.
[ x] My code changes have been verified by automated tests and pass all relevant test scenarios.

trunk-io[bot] commented 8 months ago

🚫 This pull request was closed and has been removed from the merge queue (details).

InnocentBug commented 8 months ago

@duboyal please add tests to this as discussed. Even if the tests don't work, no worries just add them anyways, and I can debug.

InnocentBug commented 8 months ago

we can't merge it yet, since it is untested

InnocentBug commented 8 months ago

I think the tests show green now.

InnocentBug commented 8 months ago

/trunk merge