flake8
passed.
JSHint
passed.
Review Request #12080 — Created Feb. 23, 2022 and submitted
The boilerplate that I used for all the callbacks-to-promises
conversions works in almost every case, but theready()
method is a
bit special because the callback name isready
instead ofsuccess
.
ThepromiseToCallbacks
adapter handled this, but the call sites inside
of theready
methods weren't testing for that callback name, so it
wouldn't work properly when calling in the old style.
Used an extension that called
ready()
on a draft review using the
callbacks style instead of promises. Saw that it worked correctly.
Summary | ID |
---|---|
0ef57e60b1d3fd861fba5b085b346f8bc97c27ab |