Skip to content

chore: Modify workflow to add PR's author and URL to the event payload - #965

Merged
Łukasz Ciesielski (lookasc) merged 1 commit into
mainfrom
chore/dev-hub
Nov 11, 2025
Merged

Łukasz Ciesielski (lookasc) merged 1 commit into
mainfrom
chore/dev-hub

Conversation

@lookasc

@lookasc Łukasz Ciesielski (lookasc) commented Nov 11, 2025 •

Copy link
Copy Markdown
Contributor

Changed the workflow in .github/workflows/developer-hub.yml to let the phrase/developer-hub repository mention and assign the author of the original merged PR.

Relates to:
https://github.com/phrase/developer-hub/pull/99

@jablan jablan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Interesting! Does the resulting PR in developer hub need to be manually approved/merged?

@lookasc

Copy link
Copy Markdown
Contributor Author

Interesting! Does the resulting PR in developer hub need to be manually approved/merged?

jablan Yes, resulting PR will still need to be manually approved and merged. The difference is that the user will be notified and assigned to the resulting PR, so we'll (hopefully) avoid plenty of forgotten PRs.
Please have a look here as well.

@jablan

jablan commented Nov 11, 2025

Copy link
Copy Markdown
Contributor

Łukasz Ciesielski (@lookasc) is there a way to automate that? there's no reason (other than a technical one) not to merge it without developer interaction.

@lookasc
Łukasz Ciesielski (lookasc) merged commit 0312fe3 into main Nov 11, 2025
13 checks passed
@lookasc
Łukasz Ciesielski (lookasc) deleted the chore/dev-hub branch November 11, 2025 14:23
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.

2 participants