Zachary Hamm

Results 6 comments of Zachary Hamm

@jkeiser had to update a test since this PR removed the experimental RemoveNode updates

@JakeGinnivan we just need to test this out and make sure the changes still work on linux. But looks good to me so far!

@JakeGinnivan sorry for losing track of this one. I have tested it on linux and it builds, so we should be good to go. Can you rebase this with the...

> @zacharyhamm I've rebased. > > I've gone through it all again, I think the fixup was the wrong approach, I've removed. Here is some things i've noticed which having...

(nevermind, it passes the change_set_approval_flow test)