Skip to content

Revise SKILL.md for clarity and validation steps#1746

Open
yoelcommit wants to merge 1 commit into
github:stagedfrom
yoelcommit:patch-2
Open

Revise SKILL.md for clarity and validation steps#1746
yoelcommit wants to merge 1 commit into
github:stagedfrom
yoelcommit:patch-2

Conversation

@yoelcommit
Copy link
Copy Markdown

@yoelcommit yoelcommit commented May 18, 2026

Updated SKILL.md to reflect new folder structure and validation checklist.

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • I have read and followed the Guidance for submissions involving paid services.
  • My contribution fixes a skill, or workflow file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, agent, skill, or workflow with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.
  • I am targeting the staged branch for this pull request.

Description


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New agent file.
  • New plugin.
  • New skill file.
  • New agentic workflow.
  • Update to existing instruction, prompt, agent, plugin, skill, or workflow.
  • Other (please specify):

Additional Notes

Consulted GitHub Copilot Code Review for this fix.


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.
@all-contributors add @yoelcommit for Skills

Updated instructions for creating skill folder and validation checklist.
Copilot AI review requested due to automatic review settings May 18, 2026 07:25
@yoelcommit yoelcommit requested a review from aaronpowell as a code owner May 18, 2026 07:25
@github-actions github-actions Bot added the skills PR touches skills label May 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Skill Validator Results

✅ All checks passed

Scope Checked
Skills 1
Agents 1
Total 2
Severity Count
--- ---:
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 0

Summary

Level Finding
ℹ️ Found 1 skill(s)
ℹ️ [make-skill-template] 📊 make-skill-template: 1,228 BPE tokens [chars/4: 1,326] (detailed ✓), 15 sections, 5 code blocks
ℹ️ ✅ All checks passed (1 skill(s))
Full validator output ```text Found 1 skill(s) [make-skill-template] 📊 make-skill-template: 1,228 BPE tokens [chars/4: 1,326] (detailed ✓), 15 sections, 5 code blocks ✅ All checks passed (1 skill(s)) ```

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 make-skill-template skill documentation to reflect an updated intended skill folder location and to replace a single command-based validation step with an inline validation checklist.

Changes:

  • Updated the “Create the Skill Directory” example path to .github/skills/<skill-name>/.
  • Replaced “Validate with npm run skill:validate” with “Validate your skill using the checklist below”.
Comments suppressed due to low confidence (1)

skills/make-skill-template/SKILL.md:125

  • Step 5 no longer mentions the repo’s automated validator. For contributions to awesome-copilot, the documented required step is running npm run skill:validate before npm run build (AGENTS.md / CONTRIBUTING.md). Consider adding that as an explicit item in this checklist (or a short “If you’re submitting to this repo…” note), otherwise contributors may rely on the checklist alone and miss CI-required validation.
   - Write a keyword-rich `description:`
   - Replace body content with your instructions
4. Add bundled resources as needed
5. Validate your skill using the checklist below

Comment on lines 27 to 31
Create a new folder with a lowercase, hyphenated name:

```
skills/<skill-name>/
.github/skills/<skill-name>/
└── SKILL.md # Required
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skills PR touches skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants