PR comment commands #24911
Replies: 2 comments
-
We could have a concept of 'auto' permissions which means:
We should avoid We could try to intelligently cache the list of collaborators, e.g. if we have a previous list of collaborators cached, and the user requesting the action is on that list, and the action is non-destructive, then we simply let it happen without refreshing the list. Mend's hosted apps could potentially add a 👀 emoji response when a command is received, so the user knows that a job is about to run, and then a 👍 or 🚀 once it's done. |
Beta Was this translation helpful? Give feedback.
-
I then review the updated PR and learn more about renovate.json syntax by looking at the bot's own change. The PR may or may not still have an actual version update. I complete the PR which now conforms to my requirements. This would mean I no longer have to push many commits to my default branch to change renovate.json manually until I see that it has the intended effect on the original version updating PR, because I have confidence that the bot-generated change to renovate.json does what I asked, as I can see its effect applied to the PR itself. Bonus points: if renovate implements this using AI, it can advertise itself as an AI-powered bot. |
Beta Was this translation helpful? Give feedback.
-
What would you like Renovate to be able to do?
Allow authorized users to control certain functionality using PR comments
If you have any ideas on how this should be implemented, please tell us here.
There exists some prior art:
And plenty of prior requests, of which #1355 is the only one I could find open.
While neither of those services explicitly mentions it, one would hope that the command must be by a maintainer and not just any random person with access to the repo (which would mean any GitHub user for public repos).
Q1: How can we know which users are allowed to make such comments. Do we need extra permissions compared to today?
Q2: Which commands do we want to support?
Q3: What are the performance impacts of both of the above (e.g. fetching allowed users each run, fetching every PR comment)
Beta Was this translation helpful? Give feedback.
All reactions