zed-gdscript icon indicating copy to clipboard operation
zed-gdscript copied to clipboard

Add support for linting warnings from godot-gdscript-toolkit

Open snorreks opened this issue 1 year ago • 5 comments

I'm submitting a...

  • [ ] Bug report.
  • [x] Feature request.

Feature request

It would be great to see the linting warnings for godot-gdscript-toolkit gdlint directly in the editor. I have seen it work in neovim and vscode. This uses the same project as the request in #18.

snorreks avatar Aug 23 '24 11:08 snorreks

Doesn't it show to you?

image

If I hover over the line, it shows the details.

Although I have godot-gdscript-toolkit gdlint installed, it works without it. That's an LSP functionality if I'm not mistaken.

gerep avatar Oct 11 '24 19:10 gerep

@gerep I believe what you're seeing there is the LSP warning that the delta variable is unused, which is provided by the LSP but the linter provides other warnings that the LSP can't and it would be useful to have those in the editor as well if possible.

moomerman avatar Oct 31 '24 21:10 moomerman

Also, it would be great to have gdformat from gdtoolkit as well

reisaraujo-miguel avatar Mar 09 '25 20:03 reisaraujo-miguel

@reisaraujo-miguel You can set up gdformat as an external tool like below in the zed settings

"languages": {
    "GDScript": {
      "hard_tabs": true,
      "formatter": {
        "external": {
          "command": "gdformat",
          "arguments": ["-"]
        }
      }
    }
  },

PranavSK avatar Mar 19 '25 11:03 PranavSK

As far as I know, it's not possible for us to implement something like this at the moment with Zed extensions. When this is possible I'd be keen on adding support for TranquilMarmot's rewrite of GDLint in Rust, as it runs instantly: it's part of our new GDScript Formatter

NathanLovato avatar Oct 16 '25 08:10 NathanLovato