-
Notifications
You must be signed in to change notification settings - Fork 275
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix a bug where oldNode was null on every step #79
Conversation
yeah, I think this would fix it. Thanks for figuring that out! I was trying to remember why the So instead of this change, can you just delete the
(or something like that) How's that sound? |
690f871
to
cc10773
Compare
Should be good to merge! |
@@ -44,8 +44,8 @@ function stepThrough(node, debug=false) { | |||
logSteps(nodeStatus); | |||
} | |||
steps.push(removeUnnecessaryParensInStep(nodeStatus)); | |||
nodeStatus.reset(); | |||
nodeStatus = step(nodeStatus.newNode); | |||
const nextNode = Status.resetChangeGroups(nodeStatus.newNode); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merge your change to run tests+linter before committing ;D
you need to import Status
here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
np :) lemme know when it's ready!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be now! I had forgotten to push yesterday...
cc10773
to
f6f1a40
Compare
Should fix #77. As far as I could see there was no point setting
oldNode
tonull
inStatus.reset
. The tests are still passing so I assume I didn't do something completely silly by removing it. Fingers crossed.