Skip to content

Use commit author instead of pusher to calculate the track time when possible#37392

Draft
lunny wants to merge 2 commits intogo-gitea:mainfrom
lunny:lunny/fix_18920
Draft

Use commit author instead of pusher to calculate the track time when possible#37392
lunny wants to merge 2 commits intogo-gitea:mainfrom
lunny:lunny/fix_18920

Conversation

@lunny
Copy link
Copy Markdown
Member

@lunny lunny commented Apr 23, 2026

Fix #18920


Generated by coding agent with Codex 5.2

@lunny lunny added the type/bug label Apr 23, 2026
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 23, 2026
@lunny lunny marked this pull request as ready for review April 23, 2026 22:25
Copy link
Copy Markdown
Contributor

@wxiaoguang wxiaoguang left a comment

Choose a reason for hiding this comment

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

I think I have asked you clearly: are you sure the email address from commit can be trusted?

Could people forge fake activities on a dead account?

@GiteaBot GiteaBot added lgtm/blocked A maintainer has reservations with the PR and thus it cannot be merged and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Apr 24, 2026
@wxiaoguang wxiaoguang marked this pull request as draft April 24, 2026 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/blocked A maintainer has reservations with the PR and thus it cannot be merged type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Time tracked in issue through PR hook assigned to merging user

3 participants