Closed gaocegege closed 6 years ago
For these suggestions about improvements, I have opened some issues to keep track. And I will address other comments today or tomorrow.
@ddysher Addressed, PTAL
Thanks! btw, can u add vendor
?
/lgtm
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ddysher To fully approve this pull request, please assign additional approvers. We suggest the following additional approvers:
Assign the PR to them by writing /assign
in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here
First PR without OWNERS file, manually merge.
/assign @ddysher