Skip to content

Conversation

@MiguelElGallo
Copy link
Contributor

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • My contribution adds a new instruction, prompt, or chat mode 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, or chat mode with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.

Description

An update to an existing skill.


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New chat mode file.
  • New collection file.
  • [] New skill file.
  • Update to existing instruction, prompt, chat mode, collection or skill.
  • Other (please specify): Update to an existing skill

Additional Notes

Added some paragraphs to the .md file, for better guidance to the agent.


By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

Copilot AI review requested due to automatic review settings January 18, 2026 12:04
Copy link
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

This PR enhances the Snowflake semantic views skill by adding guidance for data sampling to better detect joins and relationships between tables. The changes include renaming the skill from snowflake-semanticview to snowflake-semantic-views and adding instructions for sampling data, validating with queries, and improving the workflow steps.

Changes:

  • Added data sampling step (step 5) with SELECT DISTINCT queries limited to 1000 rows for discovering relationships and data types
  • Added example query syntax (step 10) for validating the final semantic view works correctly
  • Renamed the skill from snowflake-semanticview to snowflake-semantic-views in both the SKILL.md name field and README table
  • Renumbered subsequent workflow steps to accommodate the new additions

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
skills/snowflake-semanticview/SKILL.md Updated skill name, added data sampling step, added validation query example, and renumbered workflow steps
docs/README.skills.md Updated skill name reference in the skills table to match new name

MiguelElGallo and others added 3 commits January 18, 2026 14:10
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@MiguelElGallo MiguelElGallo marked this pull request as draft January 18, 2026 12:12
@MiguelElGallo MiguelElGallo marked this pull request as ready for review January 18, 2026 12:12
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