Make webkitscmpy's representation of a commit compatible with what the results database uses.
<rdar://problem/75111000>
Created attachment 422418 [details] Patch
Comment on attachment 422418 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=422418&action=review > Tools/Scripts/libraries/webkitscmpy/webkitscmpy/scm_base.py:43 > + def __init__(self, dev_branches=None, prod_branches=None, contributors=None, id=None): Worth mention that id is a built-in, but seems a reasonable variable name here.
(In reply to dewei_zhu from comment #3) > Comment on attachment 422418 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=422418&action=review > > > Tools/Scripts/libraries/webkitscmpy/webkitscmpy/scm_base.py:43 > > + def __init__(self, dev_branches=None, prod_branches=None, contributors=None, id=None): > > Worth mention that id is a built-in, but seems a reasonable variable name > here. Yes, I usually try to avoid using built-ins, but this seems like the right variable name.
Created attachment 422576 [details] Patch for landing
Committed r274080: <https://commits.webkit.org/r274080> All reviewed patches have been landed. Closing bug and clearing flags on attachment 422576 [details].