lgtm, one minor of note
https://codereview.appspot.com/7064060/diff/5001/app/store/env.js File app/store/env.js (right):
https://codereview.appspot.com/7064060/diff/5001/app/store/env.js#newcode296 app/store/env.js:296: * callback. The invocation of this command returns nothing. the doc string should also have a note that annotations come back in the delta stream so explicit fetching should be rarely be needed or unesc.
https://codereview.appspot.com/7064060/
« Back to merge proposal
lgtm, one minor of note
https:/ /codereview. appspot. com/7064060/ diff/5001/ app/store/ env.js
File app/store/env.js (right):
https:/ /codereview. appspot. com/7064060/ diff/5001/ app/store/ env.js# newcode296 env.js: 296: * callback. The invocation of this command
app/store/
returns nothing.
the doc string should also have a note that annotations come back in the
delta stream so explicit fetching should be rarely be needed or unesc.
https:/ /codereview. appspot. com/7064060/