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
Resolve #8696 #8845
Resolve #8696 #8845
Conversation
💵 To receive payouts, sign up on Algora, link your Github account and connect with Stripe/Alipay. |
@BijenderKumar1 Could you add some tests, please? |
…HS variable updates in orElse Test Suite
@guizmaii @hearnadam I have added a new test to make sure that the variable changes on the LHS of an orElse construct, trigger a retry. I have also made sure that all the existing tests pass. Thanks! |
@jdegoes @hearnadam Just a gentle reminder! Thanks! |
@jdegoes @hearnadam Just a gentle reminder to review this. Sorry to bother you. |
@jdegoes Thanks for approving the changes. Just a gentle reminder to please merge the PR as well, if possible? Thanks! 🙏 |
This PR resolves #8696, which happens as the journal reset does not happen accurately, and instead empties the journal. I resolved this issue with the
prepareJournalReset
function.I tested using the same code provided in the issue #8696, and it no longer hangs.
Thanks!
/claim #8696