Skip to content

treesitter: parsers -> grammarPlugins#1324

Merged
NotAShelf merged 1 commit intoNotAShelf:mainfrom
HeitorAugustoLN:parsers-to-grammarPlugins
Jan 8, 2026
Merged

treesitter: parsers -> grammarPlugins#1324
NotAShelf merged 1 commit intoNotAShelf:mainfrom
HeitorAugustoLN:parsers-to-grammarPlugins

Conversation

@HeitorAugustoLN
Copy link
Contributor

For compatibility with 25.11, as reported by: #1312 (comment)

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • My changes fit guidelines found in hacking nvf
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • I have added a section in the manual
    • (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
    • .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

@github-actions
Copy link

github-actions bot commented Jan 8, 2026

🚀 Live preview deployed from 3178774

View it here:

Debug Information

Triggered by: HeitorAugustoLN

HEAD at: parsers-to-grammarPlugins

Reruns: 1832

@NotAShelf
Copy link
Owner

As a non-blocking question, is there a reason we're avoiding builtGrammars? I couldn't find anything on this in the PR but I'll admit that I haven't checked too carefully.

@NotAShelf NotAShelf merged commit 17c17cf into NotAShelf:main Jan 8, 2026
5 checks passed
@HeitorAugustoLN HeitorAugustoLN deleted the parsers-to-grammarPlugins branch January 8, 2026 19:18
@HeitorAugustoLN
Copy link
Contributor Author

is there a reason we're avoiding builtGrammars?

I have explained it here: #1315 (comment)

@NotAShelf
Copy link
Owner

Got it, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants