Skip to content

quest: drop the retired stats-ticker link from the recut - #8095

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

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

Conversation

@pullfrog

@pullfrog pullfrog Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

quest/m2/recut-held.md still required quest/m2/probe/stats-ticker.md. #8021 deletes that quest, so the link dangles and quest check fails on its head:

quest: quest/m2/recut-held.md:34: link does not resolve: /quest/m2/probe/stats-ticker.md
quest: quest/m2/recut-held.md:34: requires /quest/m2/probe/stats-ticker.md, which is not a quest document; make an outside condition its own quest

This drops the link, matching what #8021 already did to quest/m2/probe/README.md, and what #8074 did for browser-send-estimate.md (its patch is already on #8021's head).

It is a separate PR because the merge queue is 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 this just drops a finished quest from recut-held.md's Required 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) | 𝕏

#8021 deletes quest/m2/probe/stats-ticker.md, so recut-held.md's Required
entry for it would dangle and quest check fails. The merge queue already
holds #8021, and GitHub refuses any push to a queued branch, so this
companion PR applies the one-line removal.

@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 b08b4ad09caf81c3b5da401dee9a87dae27b2806 against base a57ce3f9989f86903fa4e46dafafcfd5a503aa2a.

No actionable findings in this one-line change. Removing the extra stats-ticker.md dependency from quest/m2/recut-held.md:34 (base line) addresses the dangling link introduced when #8021 deletes that quest. The ticker remains listed by its own epic on this branch, and the recut's plan still explicitly requires checking outstanding claims before moving an epic.

Direction: the narrow link cleanup is consistent with #8021. At review time, both main and #8021's inspected head still contain the removed bullet, so this fix is not yet redundant there.

Verification limits: GitHub source review only, covering the complete diff, surrounding recut/epic documents, and #8021's quest changes. I did not run quest context, quest check, or a repository-wide link sweep. The Check workflow for this head was still in progress when inspected; this is not a CI validation.

@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)

@kixelated kixelated closed this Nov 6, 2026
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