Skip to content

Add embeddedtemplate alias for embedded_template - #4

Closed
Pringled wants to merge 1 commit into
mainfrom
fix-embeddedtemplate-alias
Closed

Pringled wants to merge 1 commit into
mainfrom
fix-embeddedtemplate-alias

Conversation

@Pringled

@Pringled Pringled commented Aug 1, 2026

Copy link
Copy Markdown
Member

No description provided.

.erb files are commonly labeled 'embeddedtemplate' (no underscore) by
downstream consumers, e.g. semble's file-extension table, but the bundled
grammar is keyed as 'embedded_template' to match tree-sitter's canonical
name. Without this alias, get_language/get_parser raised LanguageNotFoundError
for a grammar that's actually bundled, silently degrading .erb handling to
line-based fallback. Bumps version to 0.1.2.
@greptile-apps

greptile-apps Bot commented Aug 1, 2026

Copy link
Copy Markdown

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The new alias is resolved through the shared loader path used by both public lookup APIs, its canonical target is bundled, and the version bump feeds directly into package metadata.

Reviews (1): Last reviewed commit: "Add embeddedtemplate alias for embedded_..." | Re-trigger Greptile

@Pringled

Pringled commented Aug 1, 2026

Copy link
Copy Markdown
Member Author

Combined into #7

@Pringled Pringled closed this Aug 1, 2026
@Pringled
Pringled deleted the fix-embeddedtemplate-alias branch August 1, 2026 11:05
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