Enhancement: detect manual labeling

[thaJeztah]

Per discussion on moby/moby#28143 (comment), we should have a way to detect if labels on an issue were manually corrected. This may be a difficult one, but initial thoughts;

• add a label "manual-override" (e.g.) that tells poule to not apply labels
• detect if the issue/PR has a "person X removed label xyz" action, and if so, don't re-apply that label (may be expensive)
• detect if labels of a specific kind are already present (e.g., if a "area/xyz" label is already present, don't add a different area, likewise for "kind/xyz")
• other suggestions?

/cc @icecrime @ehazlett

[ehazlett]

those all sound reasonable to me

[icecrime]

Just one small note: as it is configured today in Docker, poule will only add labels when an issue is initially opened, reopened, or edited. Unless one of those events apply, we can manually change them and they will state.

add a label "manual-override" (e.g.) that tells poule to not apply labels

Easy enough.

detect if the issue/PR has a "person X removed label xyz" action, and if so, don't re-apply that label (may be expensive)

I don't think that's doable through GitHub's API (there's no audit trail of who did what).

detect if labels of a specific kind are already present (e.g., if a "area/xyz" label is already present, don't add a different area, likewise for "kind/xyz")

I merged something in this morning that allows just that! I added the support for filtering in serve mode, so we should be able to do things like:

- triggers:
      pull_request: [ opened ]
  operations:
      - type:       label
        filters:     {
            ~labels: [ "status/1-design-review", "status/2-code-review", "status/3-docs-review" ],
        settings: {
            patterns: {
                status/0-triage:     [ ".*" ],
            }
        }

[thaJeztah]

Sounds good already @icecrime! I suggest to monitor how it works out, and see if we need changes or not. If it works fine, I'll close this issue

Great work!