chore(issues): archive completed issue specs #1507, #1726, #1786, #1904, #1906, #1907 to closed/#1919
Merged
josecelano merged 2 commits intoJun 18, 2026
Conversation
…, torrust#1786, torrust#1904, torrust#1906, torrust#1907 to closed/ Moved 6 completed issue specs from docs/issues/open/ to docs/issues/closed/, updated their frontmatter (status: done, spec-path: closed/, last-updated-utc), and reflected the changes in the 1840 EPIC subissue table. Enhanced the cleanup-completed-issues skill with branch creation, frontmatter update, and epic parent update steps.
There was a problem hiding this comment.
Pull request overview
Archives completed issue specs from docs/issues/open/ to docs/issues/closed/, updates their frontmatter to reflect the closed state, refreshes the PR-workflow-performance EPIC’s subissue table, and extends the “cleanup completed issues” skill instructions to cover the full process.
Changes:
- Moved 6 completed issue specs (and one multi-file spec directory) from
docs/issues/open/→docs/issues/closed/and updated frontmatter (status,spec-path,last-updated-utc). - Updated EPIC #1840 subissue table rows to point at closed specs and mark several subissues as
DONE. - Enhanced the cleanup skill doc (v1.1 → v1.2) with branching + frontmatter + EPIC-update guidance.
Reviewed changes
Copilot reviewed 9 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| docs/issues/open/1840-improve-pr-workflow-performance-epic/EPIC.md | Updates EPIC subissue table to reflect closed spec paths and DONE statuses. |
| docs/issues/closed/1507-review-localhost-peer-ip.md | Marks #1507 spec as done and updates closed-path frontmatter. |
| docs/issues/closed/1786-tighten-lint-config.md | Marks #1786 spec as done and updates closed-path frontmatter. |
| docs/issues/closed/1904-1669-si-24-relocate-http-server-test-environment.md | Marks #1904 spec as done and updates closed-path frontmatter. |
| docs/issues/closed/1906-1669-si-25-relocate-udp-server-test-environment.md | Marks #1906 spec as done and updates closed-path frontmatter. |
| docs/issues/closed/1907-1669-si-26-remove-udp-protocol-peer-id-re-export.md | Marks #1907 spec as done and updates closed-path frontmatter. |
| docs/issues/closed/1726-1840-workflow-performance-sccache/ISSUE.md | Marks #1726 spec as done and updates closed-path frontmatter; includes task/AC updates. |
| docs/issues/closed/1726-1840-workflow-performance-sccache/compile-hotspot-analysis.md | Updates related-artifacts link to point to the closed spec path. |
| docs/issues/closed/1726-1840-workflow-performance-sccache/sccache-a-b-report.md | Adds local A/B benchmark report artifact. |
| docs/issues/closed/1726-1840-workflow-performance-sccache/Q-and-A.md | Adds a reviewer Q&A log file for the research issue. |
| docs/issues/closed/1726-1840-workflow-performance-sccache/experiment-results-gha.md | Adds GHA bare-build experiment results artifact. |
| docs/issues/closed/1726-1840-workflow-performance-sccache/experiment-docker-gha-results.md | Adds Docker-on-GHA experiment results artifact. |
| .github/skills/dev/planning/cleanup-completed-issues/SKILL.md | Updates skill instructions to include branch creation + frontmatter/EPIC update steps and directory-move example. |
Comments suppressed due to low confidence (7)
docs/issues/closed/1507-review-localhost-peer-ip.md:10
last-updated-utcis set to2026-06-18 18:00, which is in the future relative to the PR timestamp (2026-06-18T08:17Z). Please use a non-future UTC timestamp.
docs/issues/closed/1726-1840-workflow-performance-sccache/ISSUE.md:10last-updated-utcis set to2026-06-18 18:00, which is in the future relative to the PR timestamp (2026-06-18T08:17Z). Please use a non-future UTC timestamp.
docs/issues/closed/1726-1840-workflow-performance-sccache/ISSUE.md:384- The Task 3a acceptance-criteria bullet is missing spacing after the bold text and has a trailing comma, which makes the line render awkwardly in Markdown.
docs/issues/closed/1786-tighten-lint-config.md:10 last-updated-utcis set to2026-06-18 18:00, which is in the future relative to the PR timestamp (2026-06-18T08:17Z). Please use a non-future UTC timestamp.
docs/issues/closed/1904-1669-si-24-relocate-http-server-test-environment.md:11last-updated-utcis set to2026-06-18 18:00, which is in the future relative to the PR timestamp (2026-06-18T08:17Z). Please use a non-future UTC timestamp.
docs/issues/closed/1906-1669-si-25-relocate-udp-server-test-environment.md:11last-updated-utcis set to2026-06-18 18:00, which is in the future relative to the PR timestamp (2026-06-18T08:17Z). Please use a non-future UTC timestamp.
docs/issues/closed/1907-1669-si-26-remove-udp-protocol-peer-id-re-export.md:11last-updated-utcis set to2026-06-18 18:00, which is in the future relative to the PR timestamp (2026-06-18T08:17Z). Please use a non-future UTC timestamp.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 13 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (7)
docs/issues/closed/1507-review-localhost-peer-ip.md:10
last-updated-utcis set to a future time relative to the PR timestamp (current UTC is 2026-06-18 09:11). Please set it to a non-future UTC time.
docs/issues/closed/1786-tighten-lint-config.md:10last-updated-utcis set to a future time relative to the PR timestamp (current UTC is 2026-06-18 09:11). Please set it to a non-future UTC time.
docs/issues/closed/1904-1669-si-24-relocate-http-server-test-environment.md:11last-updated-utcis set to a future time relative to the PR timestamp (current UTC is 2026-06-18 09:11). Please set it to a non-future UTC time.
docs/issues/closed/1906-1669-si-25-relocate-udp-server-test-environment.md:11last-updated-utcis set to a future time relative to the PR timestamp (current UTC is 2026-06-18 09:11). Please set it to a non-future UTC time.
docs/issues/closed/1907-1669-si-26-remove-udp-protocol-peer-id-re-export.md:11last-updated-utcis set to a future time relative to the PR timestamp (current UTC is 2026-06-18 09:11). Please set it to a non-future UTC time.
docs/issues/closed/1726-1840-workflow-performance-sccache/ISSUE.md:10last-updated-utcis set to a future time relative to the PR timestamp (current UTC is 2026-06-18 09:11). Please set it to a non-future UTC time.
docs/issues/closed/1726-1840-workflow-performance-sccache/ISSUE.md:384- There’s a formatting glitch here: the bold sentence runs into the action name (missing space) and ends with a trailing comma. This renders awkwardly in Markdown.
…torrust#1851, torrust#1852, torrust#1868 These three spec files were already in docs/issues/closed/ but still had stale frontmatter (status: open, spec-path: open/). Updated to status: done, spec-path: closed/, and bumped last-updated-utc.
Member
Author
|
ACK ab2e4bb |
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
Archives 6 completed issue specs from
docs/issues/open/todocs/issues/closed/as part of ongoing issue documentation maintenance.Changes
Moved to
docs/issues/closed/1507-review-localhost-peer-ip.md1726-1840-workflow-performance-sccache/(directory, 6 files)1786-tighten-lint-config.md1904-1669-si-24-relocate-http-server-test-environment.md1906-1669-si-25-relocate-udp-server-test-environment.md1907-1669-si-26-remove-udp-protocol-peer-id-re-export.mdFrontmatter updates
For each moved file: set
status: done, updatedspec-pathtodocs/issues/closed/..., and bumpedlast-updated-utc.EPIC update
Updated
1840-improve-pr-workflow-performance-epic/EPIC.mdto reflect closed paths andDONEstatus for subissues #1851, #1852, #1854, #1868, #1869, and #1726.Skill enhancement
Enhanced
.github/skills/dev/planning/cleanup-completed-issues/SKILL.md(v1.1 → v1.2) with:Testing
All pre-commit checks passed: machete, linter all, doc tests.