-
Notifications
You must be signed in to change notification settings - Fork 15
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
Idea: @conda-bot please add news
#673
Comments
This is a fine idea and shouldn't be too hard to implement. |
This possible new workflow was extensively researched and tested, but it appears that we have hit a blocker; @conda-bot does not have the required permissions to push files to PRs, and this isn't something that we want to enable. There needs to be a way for the workflow to access pull request/working branch information, and this work seems a bit more complicated than we initially thought it would be. The draft PR has been closed for now due to this blocker, but if anyone has ideas for how to implement this GHA workflow, they can reference that draft pull request to get some ideas and possibly build on top of it! |
while we cannot get this idea to work for @conda-bot, could we enable the workflow to run as whomever triggers it? if a maintainer leaves the comment it should work? or if the PR owner themselves leaves the comment it should work too? |
Even as the PR author, I wasn't able to add a file to the pull request; the workflow would pick up the PR number successfully but then push the news file directly to the |
Alternatively, couldn't we have I know part of this is to get the PR number, but wouldn't the SHA of the commit be enough for folks to to find the change and then the associated PR from the commit (I'm assuming we're not rebasing on merge, right?) I'm sure we're not the only project on GitHub that has this problem, I'm sure there's a solution somebody has that we could adopt. |
This seems relevant (Ken posted this in another conversation): #556 |
What is the idea?
I find myself having to submit a PR to get its number, then going back to the IDE, create the
news
item with the correct filename, then find the issue it closes (if any)... It feels like all this could be vastly automated.What about a workflow that responded to a PR comment like:
Then the workflow would create a
news/TEMPLATE
copy with filename<PR_NUMBER>-<PR_BRANCH_NAME>
, and commit it to the PR. Then, onegit pull
after, you can edit the news. Or do it via a review comment with a suggestion.An extended syntax could also allow to add the news from the message itself:
and take care of the references at the end of the bullet point?
Not too polished, but you get the idea I hope.
The text was updated successfully, but these errors were encountered: