-
Notifications
You must be signed in to change notification settings - Fork 24
Open
Labels
help wantedExtra attention is neededExtra attention is needed
Description
The binary-size job compiles the master branch and the PR branch, then feeds both elf files to bloaty and posts a comment in the PR with te results.
There are three major issues:
- The comment triggers notifications. It's not that important, I wish that could be turned off somehow
- A new comment is posted every time. It'd be better to update the comment if there already is one
- The comments aren't posted at all when someone else opens the PR. This has to do with permissions. You can get around it, but I don't fully know what the ramifications are.
Metadata
Metadata
Assignees
Labels
help wantedExtra attention is neededExtra attention is needed