Skip to content

Point sicario-core docs to preset README#3120

Open
SiCar10mw wants to merge 1 commit into
github:mainfrom
dfirs1car1o:codex/sicario-doc-link
Open

Point sicario-core docs to preset README#3120
SiCar10mw wants to merge 1 commit into
github:mainfrom
dfirs1car1o:codex/sicario-doc-link

Conversation

@SiCar10mw

@SiCar10mw SiCar10mw commented Jun 23, 2026

Copy link
Copy Markdown

Summary

Validation

  • python3 -m json.tool presets/catalog.community.json
  • Confirmed the linked README exists and includes specify preset add --from, specify preset add --dev, badges, template summary, and verification guidance.
  • Verified the released preset still installs with specify preset add --from https://github.com/dfirs1car1o/sicario-spec/releases/download/v0.4.0/sicario-core-0.4.0.zip in a fresh Spec Kit project.

Related: #3103

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the community preset catalog entry for sicario-core so its documentation link points to the preset-scoped README (rather than the repository root README), aligning the catalog’s user-facing docs with the preset’s actual usage guidance.

Changes:

  • Updated sicario-core.documentation to .../presets/sicario-core/README.md in the community presets catalog.
Show a summary per file
File Description
presets/catalog.community.json Repoints the sicario-core preset documentation URL to the preset README path.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

"download_url": "https://github.com/dfirs1car1o/sicario-spec/releases/download/v0.4.0/sicario-core-0.4.0.zip",
"homepage": "https://github.com/dfirs1car1o/sicario-spec",
"documentation": "https://github.com/dfirs1car1o/sicario-spec/blob/main/README.md",
"documentation": "https://github.com/dfirs1car1o/sicario-spec/blob/main/presets/sicario-core/README.md",

@mnriem mnriem 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.

Please address Copilot feedback

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.

3 participants