LGTM with a suggestion to go XXX vs "doesn't make sense" note.
https://codereview.appspot.com/11227043/diff/1/app/models/models.js File app/models/models.js (right):
https://codereview.appspot.com/11227043/diff/1/app/models/models.js#newcode919 app/models/models.js:919: // than relative which would make more sense in an export. should this be an XXX vs a note then if it doesn't make much sense as is?
https://codereview.appspot.com/11227043/
« Back to merge proposal
LGTM with a suggestion to go XXX vs "doesn't make sense" note.
https:/ /codereview. appspot. com/11227043/ diff/1/ app/models/ models. js models. js (right):
File app/models/
https:/ /codereview. appspot. com/11227043/ diff/1/ app/models/ models. js#newcode919 models. js:919: // than relative which would make more sense
app/models/
in an export.
should this be an XXX vs a note then if it doesn't make much sense as
is?
https:/ /codereview. appspot. com/11227043/