Comments (5)
Just noticed approval is only done when a pull request is opened or re-opened. So I guess it would also need to run on pull request updating.
from autoapproval.
Thanks for reporting the issue.
Can you please provide a better use case for the problem? I didn't really understand this sentence.
Because of this behaviour, rebasing a branch results in the branch not being approved anymore, while still having the label.
from autoapproval.
Sure!
So I use Dependabot and it creates a several pull requests at the same time. Autoapproval approves all those pull requests, and so whichever pull request finishes the build process first is merged automatically by Dependabot.
However, after merging that first pull request, all other pull requests are updated with master through rebasing, which removes all reviews from the pull requests.
Dependabot is then unable to merge, because there's no approval anymore and Autoapproval is not re-approving the updated pull request, so then I have to manually approve it.
from autoapproval.
Thanks for your explanation! 👍
This is a valid concern. But I doubt that would work because if PR contains the labels of the approved PR, it will not be approved one more time. Being assumed the following:
- if PR contains the labels of the approved PR it will be considered as already approved.
from autoapproval.
I know that if you update the branch from GitHub interface, it will not reset the review. Maybe you can consider that?
Another thing which could work for you is removing the label of the approved PR, so that it could be approved one more time.
from autoapproval.
Related Issues (15)
- Add tests HOT 1
- Setting up the GitHub App Bot HOT 2
- autoapprove a second time the same pull request HOT 14
- Self hosted app is not showing up HOT 2
- Check PR approval status through GitHub API
- Approved comment spammed on PR HOT 7
- Remove "Write access to code" permission HOT 3
- Multiple approved comments depending on the number of labels. HOT 7
- Service down? HOT 4
- Error: Cannot read properties of null (reading 'blacklisted_labels') HOT 5
- Deploy application to Fly
- Skip draft PRs
- Automerge not occurring as expected
- what if the PR merge require 2 approvers? HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from autoapproval.