174213530d
The old code generates `structs.Repository.Permissions` with only `access.Permission.AccessMode`, however, it should check the units too, or the value could be incorrect. For example, `structs.Repository.Permissions.Push` could be false even the doer has write access to code unit. Should fix https://github.com/renovatebot/renovate/issues/14059#issuecomment-1047961128 (Not reported by it, I just found it when I was looking into this bug) --- Review tips: The major changes are - `modules/structs/repo.go` https://github.com/go-gitea/gitea/pull/25388/files#diff-870406f6857117f8b03611c43fca0ab9ed6d6e76a2d0069a7c1f17e8fa9092f7 - `services/convert/repository.go` https://github.com/go-gitea/gitea/pull/25388/files#diff-7736f6d2ae894c9edb7729a80ab89aa183b888a26a811a0c1fdebd18726a7101 And other changes are passive. |
||
|---|---|---|
| .. | ||
| deliver.go | ||
| deliver_test.go | ||
| dingtalk.go | ||
| dingtalk_test.go | ||
| discord.go | ||
| discord_test.go | ||
| feishu.go | ||
| feishu_test.go | ||
| general.go | ||
| general_test.go | ||
| main_test.go | ||
| matrix.go | ||
| matrix_test.go | ||
| msteams.go | ||
| msteams_test.go | ||
| notifier.go | ||
| packagist.go | ||
| packagist_test.go | ||
| payloader.go | ||
| slack.go | ||
| slack_test.go | ||
| telegram.go | ||
| telegram_test.go | ||
| webhook.go | ||
| webhook_test.go | ||
| wechatwork.go | ||