Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

additions to grammarPlugins should be passed through grammarToPlugin #11

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

BirdeeHub
Copy link

@BirdeeHub BirdeeHub commented Nov 17, 2024

closes #10 (a build failure, doesnt work at all until the PR goes through)

Copy link

@PerchunPak PerchunPak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should unblock updates for multiple users, see NixOS/nixpkgs#356274

@BirdeeHub
Copy link
Author

Yeah that was the issue I was having can confirm this is the fix. Have been using my branch since I fixed it here and it works

@bandithedoge bandithedoge merged commit 7629fcf into nvim-neorg:main Nov 28, 2024
@PerchunPak
Copy link

note: tests on main fail

@BirdeeHub
Copy link
Author

BirdeeHub commented Nov 28, 2024

Ah.... Ok... Well, thats cause its looking in the wrong spot now... Didnt think about that. It works now though!

@BirdeeHub
Copy link
Author

BirdeeHub commented Dec 1, 2024

the only one failing is nvim failing to create an ftdetect for neorg plugin, not for the grammar. Im not sure that case is even actually testing the grammars but rather neorg plugin itself and nvim?

It might be ok to just remove that, but regardless its hard to be 100% sure that its because of the grammars

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.

Bug with withAllGrammars
3 participants