Re: Automating `needs-rebase` label

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Patrick,

Are we already using this "ready-to-merge" label? What does it mean
exactly? That the PR is already QA'd and just needs merged or that the
PR has no conflicts? In any case, it does not seem useful to me and
adds visual clutter to the label list.

Until recently we used that in the dashboard for highlighting PRs meeting all requirements (checks, reviews, QA, etc). Using a green label helped visually identify PRs:

image.png

I'm not sure if our label colours are always helping understand the status of a PR:

image.png

We recently stopped using the 'ready-to-merge' label and we now use a column in our project board:

image.png

So I have no issue with removing the ready-to-merge label & the automation rule in that PR. Is that ok?

> The following comment will be added: "This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved".
>
> When the PR is rebased, conflict solved and re-pushed:
>
> The `needs-rebase` label will be automatically removed (`ready-to-merge` label won't be added back though)

Otherwise, a very useful change. Thanks for your efforts on it!

You're welcome!

Kind Regards,

Ernesto
 
--
Patrick Donnelly, Ph.D.
He / Him / His
Principal Software Engineer
Red Hat Sunnyvale, CA
GPG: 19F28A586F808C2402351B93C3301A3E258DD79D

_______________________________________________
Dev mailing list -- dev@xxxxxxx
To unsubscribe send an email to dev-leave@xxxxxxx

[Index of Archives]     [CEPH Users]     [Ceph Devel]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux