[webkit-reviews] review requested: [Bug 194961] [ews-app] Remove BuilderMapping table : [Attachment 362909] Proposed patch

[bugzilla-daemon at webkit.org]

Aakash Jain <aakash_jain at apple.com> has asked  for review:
Bug 194961: [ews-app] Remove BuilderMapping table
https://bugs.webkit.org/show_bug.cgi?id=194961

Attachment 362909: Proposed patch

https://bugs.webkit.org/attachment.cgi?id=362909&action=review




--- Comment #3 from Aakash Jain <aakash_jain at apple.com> ---
Created attachment 362909

  --> https://bugs.webkit.org/attachment.cgi?id=362909&action=review

Proposed patch

> Can we add a bit more of a description here? Something along the lines of
'Store builder information with build, instead of referencing builder'
Done.

> I'd like to seem a bit more descriptive changelog, but other than that, looks
good to me.
Thanks