Skip to content

quest: drop the retired stats-ticker link from browser-send-estimate - #8074

Closed
pullfrog[bot] wants to merge 1 commit into
mainfrom
pullfrog/8021-stats-ticker-link
Closed

pullfrog[bot] wants to merge 1 commit into
mainfrom
pullfrog/8021-stats-ticker-link

Conversation

@pullfrog

@pullfrog pullfrog Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

quest/m2/probe/browser-send-estimate.md still linked to quest/m2/probe/stats-ticker.md. #8021 deletes that quest, so the link dangles and quest check fails on its head:

quest: quest/m2/probe/browser-send-estimate.md:39: link does not resolve: /quest/m2/probe/stats-ticker.md

This drops the link, matching what #8021 already did to quest/m2/probe/README.md.

It is a separate PR because the merge queue was already holding #8021, and GitHub refuses any push to a queued branch. The change is safe in either order: while stats-ticker.md still exists the link resolves and removing it just drops a finished quest from the Related list, and once #8021 lands the link must go. If #8021's queue run is fixed by the land fixer first, this PR is redundant and can be closed.

Verified with a link sweep over quest/: no unresolved absolute links remain.

Pullfrog  | View workflow run | via Pullfrog | Using deepseek-v4.1-flash-1017 (free via Pullfrog for OSS) | 𝕏

@kixelated kixelated left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated review by review (OpenAI)

Reviewed head: 092dd606d5e4743f43ecbdb9551934ab1897406e.

No actionable findings. The single deletion at quest/m2/probe/browser-send-estimate.md:39 (base-side line) removes an optional Related link to the quest that #8021 deletes. The surrounding plan and remaining Related entries are unchanged, and their repository targets exist at this head. This is an appropriate companion cleanup; the inspected #8021 diff does not yet include this deletion.

Verification limits: static GitHub review of the complete diff, surrounding document, repository tree, and #8021's relevant changes. I did not run quest context, quest check, or the full repository checks, and did not independently repeat the PR's repository-wide link sweep. Rechecked that this PR is open and non-draft at the same head, with no existing reviews, immediately before submission.

@kixelated

Copy link
Copy Markdown
Collaborator

Closing as superseded: the same patch, applied unchanged, is now on moq-dev/moq#8021, which deletes the stats-ticker quest. The maintainer picked /quest-delete for this one.

(Written by Claude Opus 5.5 with high effort)

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.

1 participant