Skip to content

Create workflow for PR draft nudge comments#5

Merged
chainchad merged 1 commit intomainfrom
dx-1167/pr-draft-nudger
Jun 20, 2025
Merged

Create workflow for PR draft nudge comments#5
chainchad merged 1 commit intomainfrom
dx-1167/pr-draft-nudger

Conversation

@chainchad
Copy link
Copy Markdown
Collaborator

@chainchad chainchad commented Jun 20, 2025

In order to encourage more PR's to be created as draft mode, this will post a comment on PR's opened as 'ready for review' (non-draft) with a suggestion to use draft in the future.

@chainchad chainchad force-pushed the dx-1167/pr-draft-nudger branch from 037fe40 to 29c2123 Compare June 20, 2025 22:22
@chainchad chainchad marked this pull request as ready for review June 20, 2025 22:24
@chainchad chainchad requested a review from a team as a code owner June 20, 2025 22:24
@chainchad chainchad changed the title Create workflow for PR draft nudge commenter Create workflow for PR draft nudge comments Jun 20, 2025
Copy link
Copy Markdown
Collaborator

@erikburt erikburt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we move the actions script of this logic into a composite action in the .github repository?

@chainchad
Copy link
Copy Markdown
Collaborator Author

@erikburt certainly could but then we're updating in two places potentially (if the interface changes). Think it's worth it though?

@chainchad
Copy link
Copy Markdown
Collaborator Author

Going to merge but if you feel strongly about separating the workflow and script, I can move it to the .github repo 👍

@chainchad chainchad merged commit 791a7d7 into main Jun 20, 2025
5 of 6 checks passed
@chainchad chainchad deleted the dx-1167/pr-draft-nudger branch June 21, 2025 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants