Skip to content

Conversation

@johnsoncodehk
Copy link
Member

No description provided.

Copy link

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 refactors Vue grammar scope names from "source.vue" to "text.html.vue" to better align with TextMate grammar conventions for HTML-based languages.

Key Changes:

  • Updated the main Vue grammar scope name in vue.tmLanguage.json
  • Updated scope name references in MDX and Markdown integration files
  • Updated package.json grammar configurations and embedded language mappings
  • Regenerated test snapshots to reflect the new scope names

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
extensions/vscode/syntaxes/vue.tmLanguage.json Updated main scopeName from "source.vue" to "text.html.vue" and internal scope references
extensions/vscode/syntaxes/mdx-vue.json Updated include references from "source.vue" to "text.html.vue"
extensions/vscode/syntaxes/markdown-vue.json Updated include references from "source.vue" to "text.html.vue"
extensions/vscode/package.json Updated all scopeName and embeddedLanguages mappings from "source.vue" to "text.html.vue"
extensions/vscode/tests/__snapshots__/grammar.spec.ts.snap Updated all test snapshot expectations to use "text.html.vue" instead of "source.vue"

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

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.

2 participants