fix(score): preserve bounded PR evidence - #172
Conversation
|
@Hsiii is attempting to deploy a commit to the hikariming's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
hi bro,我正在做主站的后端分离相关,可能要晚点处理完架构问题再来审最近的issue,看怎么把语义rebase上去 |
|
了解,是 #170 這邊的 Go migration 對嗎? 我可以等轉成 Go 之後把這邊的改動改在 Go 那邊 |
|
hi~我已经做完前后端分离重构啦,可以麻烦老师重新更新一下内容吗? |
|
恭喜!沒問題! |
8373522 to
9369243
Compare
|
@AsperforMias migrate 完成,再麻煩審核了,感謝! |
|
hi~可以修下与最新main的conflict并补下popular/project这边的mock吗? |
9369243 to
f460aa7
Compare
|
沒問題,更新了! |
|
感谢补上 high-history account 的测试。 request changes: 当前改动会把 另外, 请不要把 300 条 bounded aggregate 作为最终评分输入。正确修复需要迁移原有 durable native-merged-PR paginator:按 cursor 分页保存全部 native merged PR,完成聚合后再计算并持久化 score;未完成时返回 pending,而不是 建议新增测试:
|
|
@AsperforMias 感谢老師的详细 review。 |
Description
Accounts with ≤300 PRs currently aggregate up to 300, but one with 301 PRs skips aggregation and turns ecosystem evidence into zero. This can abruptly lower an account's score with just few new PRs.
Proposed solution to make the scoring more fair:
Validation
go test ./...