Nvim-related news — Activity since 2023-09-01

• nvim-treesitter/nvim-treesitter repo issues, pull requests, and releases

  • Feat: Add unison #5340 [tapegram]

    Attempting to add unison Appreciate this tool thanks for the help!

    • amaanq

      • amaanq: forall and do sound like @repeat equals sounds like @keyword.operator

        • amaanq: Thanks, not familiar w/ the language so I just go off of preconceived notions about certain...

      • amaanq: does let declare only functions or a variable/other stuff?

        • amaanq: yeah that's good

      • amaanq: not sure what the wrapped parenthesis indicates, there's no parent node to these two

        • amaanq: Ah I see, my bad. Were you intending to match two items at the root level? Specifying a parent...

        • amaanq: Can these kind of patterns deduct consecutive nodes, or will they always be resolved to 2...

  • refactor: remove `include-children` from queries #5226 [lucario387] NEXT

    As the injection range issue has been resolved, there isn't any reason to keep the workarounds anymore I will do...

  • fix: auto install #4911 [lewis6991] NEXT

  • injections(nix): dynamic language injection via comments #4658 [figsoda] NEXT

    continuation of #3902 since neovim/neovim#21548 is in stable requires neovim/neovim#23015 to work correctly since...

  • docs: fix typo in README.md #5341 [socherbyc]

    • ObserverOfTime

  • Invalid node type at position for language bash #5345 [Maritsu] bug

    Describe the bug Upon trying to do anything in a bash script, NVIM errors out, saying "invalid node type at position...

    • clason: Duplicate of #3092

  • feat(c_sharp): add indent query #5344 [vlada-dudr]

    I have been using it for some months now and seems to mostly work ok.

    • lucario387: Is there any reason why only switch sections with 2 cases have branches, and only the second one...

      • vlada-dudr: Because: switch (var) { case "a": foo(); break; } is one switch_section starting on case:...

      • lucario387: Couldn't it be something like ; following current indent query for C (switch_body)...

      • vlada-dudr: wouldn't it make it like this? switch (var) { case "b": case "a": foo(); break; } Thus without...

      • lucario387: it's our current version for C. If that style is not a standard for C# then you can ignore this.

      • lucario387: Your queries are fine :D. I don't think this case_switch is an issue. I have done the same for...

    • lucario387: I can see a lot of (parent body: (block ... @capture)) here. Can it be replaced with only (block ...)

      • lucario387: Personal guess: You need this because you capture the parent node with @indent.begin, which is...

      • vlada-dudr: I wish I remember the reason. But I clearly do remember that I couldn't make it work properly...

      • lucario387: At its current state, I fear this is too messy for me, you, future you and others to maintain...

      • vlada-dudr: hm, I will try what happens

      • vlada-dudr: I see why I might do it this way. You may have most of these conditionals and loops not being...

    • lucario387: Imo, Some of these, from looking at the AST, can be replaced with a single (block) capture,

      • vlada-dudr: I did the whole thing incrementally over time, so i don't remember exactly all the reasons for...

      • vlada-dudr: It is because of indenting inside of those statements, like: if ( with() && many() ...

      • lucario387: Ok, so another language like dart, swift and friends where the conditions of the if statement...

    • lucario387: Why these and not (block) @indent.begin? Is there a case where the body field is not a block and...

      • lucario387: Okay I can see better why this is declared as such. I'd still prefer to capture the node...

      • vlada-dudr: yeah, classes, structs and similar have declaration_list in their bodies, but e.g. methods and...

    • lucario387: The if_statement should be similar enough to C that I suggest you can use the C's indent queries...

      • vlada-dudr: I remember I was looking onto C's version, but I understood only like half of it, so i did this,...

    • lucario387: Using your example codeblock, I got this result with the if statement if ( with() &&...

      • vlada-dudr: hm, weird I am getting this: if ( with() && many() && conditions() ) { foo(); }

    • lucario387: Edit: Consider this obsolete.

    • vlada-dudr: Can you point, how to run the tests? I could probably add some of them to make it easier to...

    • lucario387: Can you point, how to run the tests For this PR, you'll need to add example c_sharp files to...

    • vlada-dudr: That's the part which I understood, but I have no clue how to run the thing

    • lucario387: oh, just be inside the git repo, and do a ./scripts/run_tests.sh indent/c_sharp_spec.lua and see...

    • clason: No, you also need to have plenary installed and in your standard runtimepath. (I've said it...

    • vlada-dudr: Thanks @clason, that's the missing part - I expected that test machinery would grab it itself,...

    • theHamsta: Can you add some tests? At least there should be some test files on how this is expected to work.

      • vlada-dudr: Yeah, I started writing something. But I probably don't get how it is supposed to work. I...

• nvim-tree/nvim-tree.lua repo issues, pull requests, and releases

  • feat: add api.marks.bulk.trash #2391 [pr4th4m]

    API to trash bulk files using external trash cli or similar Tested with OS: MacOS...

    • alex-courtis: Many thanks for your contribution. We do need some work: remove unnecessary refresh handle...

      • alex-courtis: No changes here, just a refactor.

      • alex-courtis: We don't need this, as trash.remove already does this.

    • alex-courtis: Many thanks for this one; trash is a useful feature.

  • fix(#2382): git watcher handles worktrees and submodules, via --absolute-git-dir when it is available #2389 [alex-courtis]

    fixes #2382 This is clunky: using yet another map for project roots is fragile and not easily maintainable. Possible...

  • feat(#1917): add diagnostic highlighting and icon placement #2396 [alex-courtis]

    fixes #1917 #1510

  • feat: add renderer.icons.diagnostic_placement #2152 [baahrens]

    Attempts to close #1510 This is my first proper PR in this project, so please let me know if this is the wrong...

    • alex-courtis: superseded by #2396

  • fix(#2127) WIP: Open files with relative paths #2329 [xiantang]

    close #2127 Open files with relative paths add experimental.open.relative_path flag

    • alex-courtis: Are relative paths working when you are testing? Are you intending to complete this task?

  • NvimTree commands are not working anymore. #2384 [HicaroD] bug

    Description Commands are not working anymore. I just updated my NvimTree an hour ago, and suddenly it stopped...

    • thatanjan: Sorry for the late reply @alex-courtis. I did a clean room setup. Looks like it is a problem...

    • thatanjan: I have fixed it finally. For some reason, removing the following code solves the problem. local...

    • alex-courtis: You are calling setup twice, the first time with an empty config. Just call it once with your...

    • alex-courtis: Looks like we are all good.

  • When bulk moving, set destination based on the directory under cursor #2395 [raine] feature request

    Describe the solution you'd like When using bulk move (bmv) to move files, it would be great if the directory on the...

    • alex-courtis: I noticed this the other day. This prompt is inconsistent with other operations e.g. paste and...

    • gegoune: I don't mind breaking changes. But if we are breaking do we want to do anything else as well? Do...

  • NvimTreeOpenedFileIcon Highlight Group #2398 [LinuxIsCool] feature request

    Why is there no NvimTreeOpenedFileIcon Highlight Group? I see NvimTreeOpenedFolderIcon highlight group. But files...

created at 2023-09-02 22:04 by dinghy 1.3.0.