fix: log the PR title first #59

Closed
earl-warren wants to merge 1 commit from earl-warren:wip-log into main
Contributor

Should anything happen (like a crash), it is best to have the PR title causing the crash displayed as soon as possible.

Should anything happen (like a crash), it is best to have the PR title causing the crash displayed as soon as possible.
fix: log the PR title first
Some checks failed
/ test (pull_request) Failing after 2m14s
9173f17ef7
Should anything happen (like a crash), it is best to have the PR title
causing the crash displayed as soon as possible.
@ -226,6 +226,7 @@ func (o *release) getPullRequests(ctx context.Context) []*ReleasePullRequest {
}
logger.Info(fmt.Sprintf("loading pull requests created after %v", ageLimit))
o.forgejo.GetPullRequests(o.repository, func(pr PullRequest) bool {
info(pr.Title, pr)
Owner

i think pr can also be null. seen it in the past!

i think pr can also be null. seen it in the past!
earl-warren closed this pull request 2025-05-02 07:33:43 +00:00
Some checks failed
/ test (pull_request) Failing after 2m14s
Required
Details

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
forgejo/release-notes-assistant!59
No description provided.