Alexandr Juncevich
Alexandr Juncevich
> @juncevich Is this ready for another round of reviews? Yes, I try to fix the review notices ASAP because they are blocking another fix. I didn’t bring it up...
@ivanzlenko No need to use Atomic for OffsetDateTime, because it thread-safe by design.
@siddhantsangwan, can you take another look at this PR?
> please let me know when you've addressed comments and this is ready for another review. @siddhantsangwan I've not understood about comments, can you explain about? PR is ready for...
@siddhantsangwan, can you take another look at this PR?
> @juncevich if you've addressed all comments and fixed tests, I can start the CI workflow. Great, thanks! Just fixed the flaky test. Everything should be OK.
@siddhantsangwan I've fixed your review notices. Could you look through the changed code? I have to fix the robot test and PR will be ready.
@siddhantsangwan robot test fixed. PR is ready for another review round.
> @juncevich Thanks for updating. However, please try not to force push changes as that makes it hard to review the updates. @siddhantsangwan I respect your point, but I can't...
While trying to resolve merge conflicts almost broke everything. Sorry.