search

mergeability / mergeable

🤖 All the missing GitHub automation 🙂 🙌
https://mergeable.us
GNU Affero General Public License v3.0
683 stars 116 forks source link

Filters should include changeset #741

Open ksrembd opened 7 months ago

ksrembd commented 7 months ago

I was trying multiple combination to run some actions only when there were changes happened in a given path of the repository. chageset is helping to see if there are any changes happened in the repository and getting bot enabled. but I was unable to trigger any other actions along with change sets

Mergeable configuration

################################################################################ version: 2 mergeable:   - when: pull_request., pull_request_review.     name: 'PR title check'     validate:       - do: or         validate:           - do: changeset             must_include:               regex: 'ipnext/arch/'           - do: and             validate:               - do: changeset                 must_include:                   regex: 'ipnext/aas/'     pass:       name: 'wip check'       validate:         - do: label           and:             - must_exclude:               regex: 'wip'               regex_flag: m               message: Remove "wip" label from the PR to merge             - must_exclude:               regex: 'do not merge'               regex_flag: m               message: Remove "do not merge" label from the PR to merge

andekande commented 4 months ago

this does a multi line search if it still does not work, consider escaping: 'ipnext\/arch\/'

    validate:
      - do: changeset
        must_include:
          regex: ['ipnext/arch/', 'ipnext/aas/']
          regex_flag: 'im'
          all: false

In your 'pass' section there is an error - you can not put a 'validate' section there. So just move the label check up to the validate section and skip 'pass' for the default handling.