search

ros-controls / ros2_control_demos

This repository aims at providing examples to illustrate ros2_control and ros2_controllers
https://control.ros.org
Apache License 2.0
392 stars 185 forks source link

[CI] Reviewer Lottery: Use contains() in if rule #401

Closed christophfroehlich closed 8 months ago

christophfroehlich commented 8 months ago

Maybe this works now, see https://docs.github.com/en/actions/learn-github-actions/expressions#example-matching-an-array-of-strings

christophfroehlich commented 8 months ago

@mergifyio backport iron

mergify[bot] commented 8 months ago

backport iron

✅ Backports have been created

* [#402 [CI] Reviewer Lottery: Use contains() in if rule (backport #401)](https://github.com/ros-controls/ros2_control_demos/pull/402) has been created for branch `iron`
christophfroehlich commented 8 months ago

I found out why it did not work earlier: https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request_target

This event runs in the context of the base of the pull request, rather than in the context of the merge commit, as the pull_request event does. This prevents execution of unsafe code from the head of the pull request that could alter your repository or steal any secrets you use in your workflow. This event allows your workflow to do things like label or comment on pull requests from forks. Avoid using this event if you need to build or run code from the pull request.