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
{{ message }}
Repository navigation
Existing comment's links should be automatically updated by the bot #24
Currently, cpp-warning-notifier does nothing when the recent warning status exactly matches the previous build. This was a design decision I proposed and is still the preferred behavior. However, I still want the links to be updated unconditionally for several reasons:
People who don't know the above behavior will get confused by the stale links
I sometimes want to click the links to use them as a "quick jump" button
I think there's an API for the bots to update their own comments.
For clarification, does "when the recent warning status exactly matches the previous build" mean "even if both builds have warnings, they differ when warning bodies differ"? It changes the warning detection logic, because it have to parse warning bodies for that.
I DO think just "both have warnings, so update links" have less sense, but parsing and comparing the warning bodies is bit complex.
P.S.
I implemented the logic, according to my assumption. Please check PR #28
Activity
yaito3014 commentedon Oct 15, 2026
For clarification, does "when the recent warning status exactly matches the previous build" mean "even if both builds have warnings, they differ when warning bodies differ"? It changes the warning detection logic, because it have to parse warning bodies for that.
I DO think just "both have warnings, so update links" have less sense, but parsing and comparing the warning bodies is bit complex.
P.S.
I implemented the logic, according to my assumption. Please check PR #28
yaito3014 commentedon Oct 15, 2026
#28 (comment)