zappr icon indicating copy to clipboard operation
zappr copied to clipboard

OneOf Labels Check

Open drummerwolli opened this issue 5 years ago • 5 comments

in the label check config, it should be possible to define an array of labels, where just one label needs to be matched for the check to be fulfilled. config could look something like this:

pull-request:
  labels:
    # pull request cannot be merged without these labels (one match is enough)
    oneOf:
      - merge to dev
      - merge to live
    # pull request cannot be merged without these labels (all need to be matched)
    required:      
      - reviewed

(this example config also demonstrates my use-case, i want to to make it mandatory to indicate which branch you want to merge to) (trying to maintain backwards compatibility, naming can be adjusted, of course)

drummerwolli avatar Jan 24 '19 09:01 drummerwolli

Would your feature request be also covered with integrating existing apps like GitHub Required Labels?

lotharschulz avatar Jan 25 '19 16:01 lotharschulz

👆 = Zalando employee

Yes, but I need this on our internal zappr running on GHE.

drummerwolli avatar Jan 25 '19 17:01 drummerwolli

Sure, connection to Zalando GHE similar to https://github.com/dimagi/required-labels#installation would be required.

lotharschulz avatar Jan 25 '19 17:01 lotharschulz

I'm not sure I understand correctly, should I use now this other tool to get this done? First I would need to deploy this then somewhere internally, and then maintain the connection to it, etc, etc. Zappr is already running internally, so as soon as this is done, it would automatically appear as a feature internally as well, for everyone at Zalando.

I see this as a fairly small feature request to the already existing label checker in zappr. Especially in comparison with that, the addition and setup of another tool sounds non-viable to me.

drummerwolli avatar Jan 25 '19 17:01 drummerwolli

I see two options for Zalando internal use case (GHE @ Zalando):

  1. implement the feature similar to the description above
  2. install tools like required-labels as part of the platform (DF) and provide such a tool as a service to zalando devs

lotharschulz avatar Jan 28 '19 07:01 lotharschulz