Attention is currently required from: foss@volatilesystems.org, Angel Pons.
View Change
1 comment:
Patchset:
Patch Set #1:
Thought it might have been better to wait for clarity on the commit message before doing that, but I […]
Usually the rule of thumb is: if you mark comments as resolved, you need to upload new patchset which demonstrates that you indeed resolved the comments. Otherwise how do we know what you have in local repo? :)
Gerrit allows to view "incremental" diffs between patchsets, which can be very handy.
Also it is entirely normal to resolve some of the comments (not all of them), and for the remaining ones ask for clarification, not a problem!
To view, visit change 69309. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Iced40403c6694a55fd648ea2785cdcba21712234
Gerrit-Change-Number: 69309
Gerrit-PatchSet: 2
Gerrit-Owner: foss@volatilesystems.org
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: foss@volatilesystems.org
Gerrit-Attention: Angel Pons <th3fanbus@gmail.com>
Gerrit-Comment-Date: Wed, 23 Nov 2022 23:07:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: foss@volatilesystems.org
Comment-In-Reply-To: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-MessageType: comment