Notify author in PR size job#447
Conversation
Added a step to comment on PR if it exceeds size limits.
Added comment to clarify permissions for PR size check.
PrinsFrank
left a comment
There was a problem hiding this comment.
Are you sure this will work? I don't think you'd have the right permissions to do so from forks, so this will probably only work for my own PRs? https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#changing-the-permissions-in-a-forked-repository
Also, won't this keep adding comments on every push as long as a PR is over the line limit if this would work?
|
I really don't know. This is an inspiration for you. |
|
I like the idea 😆 It supports your view on the size-limit of pull reuqests. |
|
@szepeviktor Thanks for the contribution, I'll close this for now as other CI jobs are also only failing and don't comment either, and I don't think this will work cleanly out of the box. Feel free to continue working on this if you'd like! |
Added a step to comment on PR if it exceeds size limits.
@PrinsFrank No need to open CI output.