Include PR Author Name in changelog #73

Open
opened 2023-03-17 02:20:57 +00:00 by techknowlogick · 3 comments

In addition to the PR #, we could also include the @ of the PR author (this may be difficult with Giteabot being the user who opens the PR though).

In addition to the PR #, we could also include the @ of the PR author (this may be difficult with Giteabot being the user who opens the PR though).
Owner

It may need a token, but could we use the commit author? (an extra N API hits to get commit info)

Not sure, though, as we squash merge.

It may need a token, but could we use the commit author? (an extra N API hits to get commit info) Not sure, though, as we squash merge.
Author
Owner

It may need a token, but could we use the commit author? (an extra N API hits to get commit info)

It's been a sec since I've been in the changelog codebase (ignoring the CI commits I did recently), but we could optionally use the token if the appropriate env var is set.

Not sure, though, as we squash merge.

May require some parsing of the commit message to get co-authors (if needed) as well.

> It may need a token, but could we use the commit author? (an extra N API hits to get commit info) It's been a sec since I've been in the changelog codebase (ignoring the CI commits I did recently), but we could optionally use the token if the appropriate env var is set. > Not sure, though, as we squash merge. May require some parsing of the commit message to get co-authors (if needed) as well.
Owner

I would also like to include the PR indexes like mozilla did https://blog.mozilla.org/community/2021/04/16/new-contributors-to-firefox/

I would also like to include the PR indexes like mozilla did https://blog.mozilla.org/community/2021/04/16/new-contributors-to-firefox/
lunny added the
feature
label 2023-03-17 04:06:35 +00:00
Sign in to join this conversation.
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: gitea/changelog#73
No description provided.