Skip to content

Automatically update stale comment links - #27

Closed
yaito3014 wants to merge 3 commits into
mainfrom
update-stale-comment-links
Closed

yaito3014 wants to merge 3 commits into
mainfrom
update-stale-comment-links

Conversation

@yaito3014

Copy link
Copy Markdown
Member

fixes #24

No behavior change. The create/replace/skip choice now lives in
action/src/comment.ts so it can be unit-tested with node --test.
When neither the new build nor the previous bot comment has a warning,
the bot used to stay silent, leaving the old comment pointing at a
stale run. It now edits that comment in place with the fresh table, so
the links always jump to the latest run while still not notifying
anyone.

Fixes #24
An object literal with only an id has no property in common with
CommentLike, so tsc rejected it (TS2559). Neither the Rollup build nor
node --test type-checks the test file, which is why it went unnoticed.
@cppwarningnotifier

Copy link
Copy Markdown

@yaito3014 yaito3014 closed this Oct 15, 2026
@yaito3014
yaito3014 deleted the update-stale-comment-links branch October 15, 2026 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Existing comment's links should be automatically updated by the bot

1 participant