-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
chore: Separate workflow to add/remove 'stale' label #7959
Conversation
210f85b
to
c9bb67b
Compare
c9bb67b
to
7f6dd21
Compare
Co-authored-by: kenjis <[email protected]>
5ab5210
to
58acfec
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if it would work without the cronjob.
But let's merge and see.
#7986 has conflict, but not labeled. |
Or maybe a commit on the PR shows it's still a conflict. I want to see where the job action has been going. |
Yes. #7967 was merged. |
The first merge returns UNKNOWN and then the value is determined. |
It's not always going to be of real value, |
I testing with schedule as already suggested by @totoprayogo1916, always revalidate limitable latest PR by get latest status mergeable when schedule running. https://github.com/PHPDevsr/check-conflict-action/actions |
I think "Run at every 6 minutes" is running too often. In the case of UNKNOWN, I think if we wait some seconds before retrieving again, we can get the result. |
Check Conflicts Remove label failed. |
Before conflict https://github.com/codeigniter4/CodeIgniter4/actions/runs/6309422686/job/17129306013 success running with read permission because no have label stale to remove using write permission. And after conflict bot trigger added label and comment using. Many action running using permission read, I think this section of permission job.
You can see 2 action running well with write permission |
@ddevsr Sorry, I don't get what you say. |
If only we could execute this action at the end instead of others. |
Removed label-add-remove-conflict-single-pr.yml, because it does not work at all. |
Description
Follow-up #7957
Checklist: