search

vmware-tanzu / velero

Backup and migrate Kubernetes applications and their persistent volumes
https://velero.io
Apache License 2.0
8.63k stars 1.39k forks source link

Add OWNERS file to allow prow auto merge #7783

Closed kaovilai closed 4 months ago

kaovilai commented 4 months ago

Add OWNERS file like so and subsequent

# List of usernames who may use /lgtm
reviewers:
- user1
- user2
- user3

# List of usernames who may use /approve
approvers:
- user1
- user2
- admin1

Add this github action to allow auto merge when lgtm is applied.

name: "Merge on lgtm label"
on:
  schedule:
  - cron: "0 * * * *"

jobs:
  execute:
    runs-on: ubuntu-latest
    steps:
      - uses: jpmcb/prow-github-actions@v1
        with:
          jobs: 'lgtm'
          github-token: "${{ secrets.GITHUB_TOKEN }}"

https://github.com/jpmcb/prow-github-actions/blob/main/docs/examples.md#automatic-pr-merger

Follow up from https://github.com/vmware-tanzu/velero/pull/7776

blackpiglet commented 4 months ago

@kaovilai Thanks for bringing this up. I will follow this issue to introduce more PROW commands.

blackpiglet commented 4 months ago

Port the comment in PR #7784 here.

The lgtm command is not included in this PR intentionally. Auto-merging seems too much for now. Let's get familiar with the PROW-style workflow first.