Skip to content

docs(issues): [#2185] close external-link triage handoff - #2258

Merged
josecelano merged 3 commits into
torrust:developfrom
josecelano:2185-close-triage-and-handoff-to-semantic-link-epic
Sep 18, 2026
Merged

josecelano merged 3 commits into
torrust:developfrom
josecelano:2185-close-triage-and-handoff-to-semantic-link-epic

Conversation

@josecelano

Copy link
Copy Markdown
Member

Closes #2185.

Summary

Validation

  • linter markdown
  • linter cspell
  • linter lychee
  • git diff --check
  • linter all
  • ./contrib/dev-tools/git/hooks/pre-commit.sh via installed pre-commit hook
  • pre-push hook

Copilot AI lite review requested due to automatic review settings September 18, 2026 08:32
@josecelano josecelano self-assigned this Sep 18, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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

Closes issue #2185 by finalizing documentation evidence for external link check triage, preserving residual failures as an EPIC handoff artifact, and marking completion/acceptance reviews as done.

Changes:

  • Updates the #2185 baseline and issue spec to reflect hosted reruns, final dispositions, and closure decision.
  • Adds an EPIC handoff artifact preserving residual Lychee failures and links it from issue review reports and the EPIC draft.
  • Extends cspell project dictionary for new GitHub anchor tokens and referenced hostnames.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
project-words.txt Adds cspell allowlist entries for new GitHub anchor terms and referenced hostname.
docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/external-link-baseline.md Records rerun evidence, expands dispositions for persistent failures, and documents post-baseline GitHub anchor class (C10).
docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/agent-review-reports.md Links new EPIC handoff artifacts and records final independent review.
docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md Marks the task as done, completes acceptance/verification sections, and adds closure decision rationale.
docs/issues/drafts/refactor-semantic-link-conventions/external-link-check-residual-failures-2026-09-18.md New artifact preserving residual failures verbatim with root-cause notes for EPIC follow-up.
docs/issues/drafts/refactor-semantic-link-conventions/EPIC.md Adds #2185 handoff context and introduces subissues/questions tying validation work to EPIC #2003 boundaries.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/issues/open/2185-2003-triage-advisory-external-link-check-findings/ISSUE.md Outdated
@josecelano

Copy link
Copy Markdown
Member Author

ACK 0624e4e

@josecelano
josecelano merged commit 94b999d into torrust:develop Sep 18, 2026
1 check passed
josecelano added a commit that referenced this pull request Sep 18, 2026
b619fbf docs(issues): archive completed external-link triage spec (Jose Celano)

Pull request description:

  Follow-up to #2258 and #2185.

  ## Summary

  - moves the completed #2185 issue specification from `docs/issues/open/` to `docs/issues/closed/`
  - updates live semantic-link and EPIC #2003 references to the closed issue-spec path
  - records PR #2258 as the related PR and marks the issue-spec lifecycle checkbox complete

  ## Validation

  - linter markdown
  - linter cspell
  - linter lychee
  - git diff --check
  - pre-commit hook
  - pre-push hook

ACKs for top commit:
  josecelano:
    ACK b619fbf

Tree-SHA512: 0e6e41e4a3c1c0f7c1955a35ed5fdecb16a6ee8eb7aabb5067e35d6ed5f2f7b373e149580eb1df1d67b0d5722ab9cd12018f16ef138c015a28a4e84096087299
@josecelano
josecelano deleted the 2185-close-triage-and-handoff-to-semantic-link-epic branch September 18, 2026 09:16
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.

Triage advisory external-link check findings

2 participants