EmbarkStudios / octobors

Rust program for automerging PRs based on a few rules
Apache License 2.0
40 stars 2 forks source link

Don't add "reviewed" label to "trivial" PRs #38

Closed davidpdrsn closed 1 year ago

davidpdrsn commented 1 year ago

Checklist

Description of Changes

Fixes https://github.com/EmbarkStudios/octobors/issues/37

The biggest change is that pr_approved now always checks whether the PR is actually approved or not. It doesn't consider trivial PRs to always be approved. This gives us more control over when to add the "reviewed" label since that previously was always added if a PR was approved.

davidpdrsn commented 1 year ago

Think I'll just close this for now. If this is still something we want we can always revisit it.