You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A compromise may be to keep the commit comment, but also add a pointer comment on the PR itself referencing the latest commit requiring review. This comment would be updated automatically to point to the new commit.
All approaches need to be inspected for possible TOCTOU attacks.
The text was updated successfully, but these errors were encountered:
Commit comments are not obvious in the PR timeline and need to be pointed out for better UX.
Upon clicking on the tiny comment emoji, you are brought to the commit view for approval.
Another approach would be to comment directly on the PR and reference the commit SHA and workflow run.
A compromise may be to keep the commit comment, but also add a pointer comment on the PR itself referencing the latest commit requiring review. This comment would be updated automatically to point to the new commit.
All approaches need to be inspected for possible TOCTOU attacks.
The text was updated successfully, but these errors were encountered: