docs(links): [#2185] record C6 hosted verification - #2248
Open
josecelano wants to merge 1 commit into
Open
josecelano wants to merge 1 commit into
josecelano wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Pull request overview
Records hosted verification evidence for the C6 Docker ACI external-link repairs related to issue #2185, while keeping remaining C6 fragments and C7/C8 work pending.
Changes:
- Updates the C6 baseline disposition to “hosted-verified” for the Docker ACI sub-slice and adds a dedicated verification section with run metrics.
- Adds an agent review report entry documenting the hosted verification inputs, findings, and verdict.
- Updates the issue tracker evidence and acceptance criteria to include the hosted run verifying the Docker repairs.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md | Marks C6 Docker repairs as hosted-verified and records detailed run-based verification evidence. |
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/agent-review-reports.md | Adds a dated reviewer entry summarizing hosted verification evidence and follow-ups. |
| docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md | Updates task status notes, evidence log, and acceptance criteria to reflect hosted verification completion for C6 Docker. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| | --- | ----------- | ----------------------------------------- | ---------------------------------------------------------------------------------------------------------------- | | ||
| | T1 | DONE | Preserve and classify the baseline | `external-link-baseline.md` maps all 461 report errors to nine recurring categories and dispositions. | | ||
| | T2 | IN_PROGRESS | Repair clearly stale references | C3-C5 hosted-verified; Docker ACI repair pending hosted verification; remaining C6 fragments pending. | | ||
| | T2 | IN_PROGRESS | Repair clearly stale references | C3-C6 Docker repairs hosted-verified; remaining C6 fragments and C7-C8 pending. | |
|
|
||
| ### C6 Docker repair verification | ||
|
|
||
| [External Link Check run 35128890381](https://github.com/torrust/torrust-tracker/actions/runs/35128890381) ran after PR #2231 merged on revision `6e1e9d29b5d88b763e1748ad45afabe5d725d62f`. The `Check External Links` step failed visibly and `Upload Lychee Report` succeeded. The retained `lychee-external-link-report` artifact is 1,229 bytes and expires on 2026-09-30. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Validation
Hosted evidence: run 35128890381 on merge commit 6e1e9d2; the retained report contains neither retired Docker URL as an error and preserves unrelated failures.
Related to #2185