Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[meta] Add automated PR governance workflows #3466

Merged
merged 5 commits into from
Jul 16, 2024

Conversation

yuxiqian
Copy link
Contributor

@yuxiqian yuxiqian commented Jul 10, 2024

This adds some GitHub workflows that:

  • automatically assigns newly opened PRs to active community contributors
  • pin reviewed & approved tags to track PR progress
  • close stale or inactive PRs

This is meant to reduce PR overall response time and increase PR reviewing & merging efficiency.

@github-actions github-actions bot added the build label Jul 10, 2024
@lvyanquan
Copy link
Contributor

Further more, can you add approve label to help PR reviewing? https://github.com/apache/seatunnel/blob/5bf6f7892a65ed26ef0bee3d6e21638bd7cab37b/.github/workflows/approve-label.yml#L48 provided an example.

@yuxiqian yuxiqian changed the title [meta] Add PR auto-assign bot [meta] Add automated PR governance workflows Jul 11, 2024
@yuxiqian
Copy link
Contributor Author

Tested in my forked repo, and reviewer auto-assigning & PR progress tagging features work as expected.

Looking forward to @GOODBOY008 @leonardBang's thoughts.

@leonardBang
Copy link
Contributor

automatically assigns newly opened PRs to active community contributors

I'm a little concerned about the auto assignment action, could we change to auto request review action instead of auto assignment ?

@yuxiqian
Copy link
Contributor Author

Removed PR auto assigning bot since more discussion is needed. Also changed Stale PR standards to 60 days inactive & 30 days before being closed.

Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @yuxiqian for the contribution, LGTM

@leonardBang leonardBang merged commit 11deb62 into apache:master Jul 16, 2024
16 checks passed
wuzhenhua01 pushed a commit to wuzhenhua01/flink-cdc-connectors that referenced this pull request Aug 4, 2024
qiaozongmi pushed a commit to qiaozongmi/flink-cdc that referenced this pull request Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants