Skip to content

Refresh _template-skill-name: align with Labs conventions + add builder-audience checklist#14

Merged
jdanielmyers merged 1 commit into
mainfrom
refresh/_template-skill-name
May 20, 2026
Merged

Refresh _template-skill-name: align with Labs conventions + add builder-audience checklist#14
jdanielmyers merged 1 commit into
mainfrom
refresh/_template-skill-name

Conversation

@jdanielmyers
Copy link
Copy Markdown
Member

Summary

Two small corrections to the contributor template:

  1. type: — lowercase to match the audit rubric and the public schema (community | snowflake | partner). The previous value Community would have failed the audit's labs.frontmatter.values check if a contributor copy-pasted it verbatim.
  2. Author checklist — fix id to name (the actual frontmatter field) and add an explicit reminder that skills here target external Snowflake builders (developers, data engineers), not internal Snowflake staff. Lists internal references that disqualify a skill (Snowhouse, PST.SVCS, MEDDPICC, etc.).

Test plan

  • Diff is minimal (3 inserts, 2 deletes); no behavioral change
  • Renders correctly on github.com (markdown)

@jdanielmyers
Copy link
Copy Markdown
Member Author

jdanielmyers commented May 20, 2026

Audit Report: _template-skill-name

Re-audit (post-v1.1.1 + critical fixes). Original report superseded.

Summary

Category 🔴 🟡 🟢
Frontmatter 0 0 1
Structure 0 0 1
Workflow 0 0 0

Critical 🔴

None — this is a template skill with placeholder content by design.

Warnings 🟡

None.

Suggestions 🟢

  1. Frontmatter name uses placeholder my-skill-name which doesn't match folder _template-skill-name → Expected for a template, but could confuse automated tooling.
  2. The prompt field still references the placeholder name $my-skill-name → Document this as intentional for copy/paste usage.

Verdict

Ready to merge — This is a template/example skill; placeholder values are intentional and instructional. Structure follows best practices correctly.

…e checklist

- Fix `type:` to lowercase (community/snowflake/partner) per audit rubric
- Update author checklist: `name` not `id`, add audience-fit reminder
@jdanielmyers jdanielmyers force-pushed the refresh/_template-skill-name branch from 50e3906 to e32f8f0 Compare May 20, 2026 21:01
@jdanielmyers jdanielmyers merged commit 3d70704 into main May 20, 2026
1 check failed
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.

1 participant