update authors [chromium/src : main]

0 views
Skip to first unread message

Tom Sepez (Gerrit)

unread,
Jun 21, 2024, 12:36:52 PM (10 days ago) Jun 21
to Johnson Adodo, chromium...@chromium.org, blundell+...@chromium.org, spang...@chromium.org, tracing...@chromium.org, wfh+...@chromium.org
Attention needed from Johnson Adodo

Tom Sepez added 3 comments

Patchset-level comments
File-level comment, Patchset 1 (Latest):
Tom Sepez . resolved

Hi, welcome, the code is fine but the commit message needs a little tidying.

Commit Message
Line 10, Patchset 1 (Latest):swap unique_ptr for HeapArray
Tom Sepez . unresolved

This should be the first sentence, e.g. the CL desciption

Swap unique_ptr<[]> for HeapArray<> in test_utils.h

Line 12, Patchset 1 (Latest):
Tom Sepez . unresolved

The follow-up should be something like

-- add self to authors file.

Open in Gerrit

Related details

Attention is currently required from:
  • Johnson Adodo
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedNo-Unresolved-Comments
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I9a3e8e68eff18224646fb229351f8624177b0e02
Gerrit-Change-Number: 5644834
Gerrit-PatchSet: 1
Gerrit-Owner: Johnson Adodo <johnso...@gitstart.dev>
Gerrit-Reviewer: Tom Sepez <tse...@chromium.org>
Gerrit-Attention: Johnson Adodo <johnso...@gitstart.dev>
Gerrit-Comment-Date: Fri, 21 Jun 2024 16:36:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages