Linux maintainer tooling and workflows
 help / color / mirror / Atom feed
* review-tui: Handle different Patchwork current statuses
@ 2026-09-24 12:52 Krzysztof Kozlowski
  2026-09-24 12:53 ` Krzysztof Kozlowski
  0 siblings, 1 reply; 2+ messages in thread
From: Krzysztof Kozlowski @ 2026-09-24 12:52 UTC (permalink / raw)
  To: Kernel.org Tools; +Cc: Konstantin Ryabitsev, Rob Herring, Conor Dooley

Hey,

I am trying to adapt b4 review to DT workflow with DT patchwork and I
think interpretation of current patch status needs customization. I have
in .gitconfig:

[b4]
	pw-review-state = needs-ack
	pw-accept-state = not-applicable
	pw-discard-state = changes-requested

But b4 review tui in Patchwork status does not show any patches under
Needs ACK.

https://patchwork.kernel.org/project/devicetree/list/

To recap, our statuses are used like:
0. "New": not processed by any automation
1. "Under Review": being processed by automation
2. "Needs ACK": Actually waiting for our review or being reviewed. This
is what we want to see in list of patches.
3. "Changes requested": Patch rejected with changes needed
4. "Not applicable": Patch was reviewed but handled elsewhere as most of
work is like that.

Either I miss some .gitconfig options or b4 is missing such
customization. If the latter, can you add a feature-request for this? I
would like to adopt b4 and move away from our internal scripting...


Best regards,
Krzysztof


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: review-tui: Handle different Patchwork current statuses
  2026-09-24 12:52 review-tui: Handle different Patchwork current statuses Krzysztof Kozlowski
@ 2026-09-24 12:53 ` Krzysztof Kozlowski
  0 siblings, 0 replies; 2+ messages in thread
From: Krzysztof Kozlowski @ 2026-09-24 12:53 UTC (permalink / raw)
  To: Kernel.org Tools; +Cc: Konstantin Ryabitsev, Rob Herring, Conor Dooley

On 24/09/2026 14:52, Krzysztof Kozlowski wrote:
> Hey,
> 
> I am trying to adapt b4 review to DT workflow with DT patchwork and I
> think interpretation of current patch status needs customization. I have
> in .gitconfig:
> 
> [b4]
> 	pw-review-state = needs-ack
> 	pw-accept-state = not-applicable
> 	pw-discard-state = changes-requested
> 
> But b4 review tui in Patchwork status does not show any patches under
> Needs ACK.
> 
> https://patchwork.kernel.org/project/devicetree/list/
> 
> To recap, our statuses are used like:
> 0. "New": not processed by any automation
> 1. "Under Review": being processed by automation
> 2. "Needs ACK": Actually waiting for our review or being reviewed. This
> is what we want to see in list of patches.
> 3. "Changes requested": Patch rejected with changes needed
> 4. "Not applicable": Patch was reviewed but handled elsewhere as most of
> work is like that.
> 
> Either I miss some .gitconfig options or b4 is missing such
> customization. If the latter, can you add a feature-request for this? I
> would like to adopt b4 and move away from our internal scripting...
> 

Ah, and if that matters: b4 v0.16.0

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-09-24 12:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-24 12:52 review-tui: Handle different Patchwork current statuses Krzysztof Kozlowski
2026-09-24 12:53 ` Krzysztof Kozlowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox