Update org-nvim tree sitter grammar#403455
Conversation
…grammars'" This reverts commit fe9912b.
Sorry about that, didn't mean to create a redundent PR. I removed all other parser updates, keeping only nvim-orgmode. Thanks for linking that discussion too. How does this look now? Should I keep the change in the comments about the NIXPKGS_GITHUB_TOKEN env var? |
|
Please squash your commits, see CONTRIBUTING.md |
|
qweered
left a comment
There was a problem hiding this comment.
Approved automatically following the successful run of nixpkgs-review.
|
sry it took so long :/ |
|
(auto merge is hanging on a GHA bug that failed to schedule one of the jobs, I can't retry it manually because it's too old, so a rebase and force push is needed) |
|
can yoiu cleanup history before merging please ? 2 commits should be enough |
There was a problem hiding this comment.
PR title and commit descriptions do not follow nixpkgs conventions, please see https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#commit-conventions
|
There has been some refacto of the treesitter infra, this now needs to be updated at pkgs/development/tools/parsing/tree-sitter/grammars/grammar-sources.nix |
Things done
I ran the update script produced from the following command
I kept only the changes made to the tree-sitter-org-nvim.json file. This pull request updates all other grammars #402457. I am only updating the org-nvim grammar here.
I also changed the comment in default.nix file. It was inaccurate about the NIXPKGS_GITHUB_TOKEN env var. It is actually referred to as GITHUB_TOKEN
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.