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

actions stale bot: Use actions/stale@v9 bot with two merged PRs to fix caching when using the action multiple times #41403

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

autoantwort
Copy link
Contributor

The bot was already updated in the past, but this was reverted because the action contained bugs. With these PRs the bugs are fixed.
This would fix "errors" like this one: #34268 (comment) since every issue is visited after some time.

…x caching when using the action multiple times
@BillyONeal
Copy link
Member

I'm concerned about the compliance flak the vcpkg team will get for enabling 3rd party actions no matter how Righteous they are. :/

@autoantwort
Copy link
Contributor Author

Fair. Maybe you can ping your colleagues from the actions team to merge the PRs instead :)

@Mengna-Li Mengna-Li added the category:infrastructure Pertaining to the CI/Testing infrastrucutre label Oct 8, 2024
@JavierMatosD JavierMatosD added depends:different-pr This PR or Issue depends on a PR which has been filed requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look. labels Oct 8, 2024
@JavierMatosD
Copy link
Contributor

I'm going to mark this PR draft until actions/stale#1169 lands. In the meantime, I'll bring this up to the team to discuss the compliance concern raised by @BillyONeal

@JavierMatosD JavierMatosD marked this pull request as draft October 8, 2024 21:54
@Cheney-W Cheney-W removed the requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look. label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:infrastructure Pertaining to the CI/Testing infrastrucutre depends:different-pr This PR or Issue depends on a PR which has been filed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants