Skip to content

Add Duso language support#5174

Open
davebalmer wants to merge 4 commits intozed-industries:mainfrom
davebalmer:main
Open

Add Duso language support#5174
davebalmer wants to merge 4 commits intozed-industries:mainfrom
davebalmer:main

Conversation

@davebalmer
Copy link

  • LSP
  • syntax highlighting

- LSP
- syntax highlighting
@cla-bot
Copy link

cla-bot bot commented Mar 10, 2026

We require contributors to sign our Contributor License Agreement, and we don't have @davebalmer on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@davebalmer
Copy link
Author

@cla-bot check

@cla-bot
Copy link

cla-bot bot commented Mar 10, 2026

We require contributors to sign our Contributor License Agreement, and we don't have @davebalmer on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot
Copy link

cla-bot bot commented Mar 10, 2026

The cla-bot has been summoned, and re-checked this pull request!

@davebalmer
Copy link
Author

@cla-bot check

@cla-bot
Copy link

cla-bot bot commented Mar 10, 2026

We require contributors to sign our Contributor License Agreement, and we don't have @davebalmer on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot
Copy link

cla-bot bot commented Mar 10, 2026

The cla-bot has been summoned, and re-checked this pull request!

@davebalmer
Copy link
Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed label Mar 10, 2026
@cla-bot
Copy link

cla-bot bot commented Mar 10, 2026

The cla-bot has been summoned, and re-checked this pull request!

Copy link
Member

@MrSubidubi MrSubidubi left a comment

Choose a reason for hiding this comment

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

Please

  • use the Worktree::which API to look for the Duso LSP binary and return an error if the binary cannot be found
  • remove the grammars directory, as this should not be part of the extension here.

@davebalmer
Copy link
Author

davebalmer commented Mar 11, 2026

OK I will address these. Sorry I missed them. thank you!

BTW I saw the CLI fail, missing license. Weird my LICENSE file is Apache 2.0 what am I missing? https://github.com/duso-org/duso-zed/blob/main/LICENSE thanks again!

@davebalmer davebalmer requested a review from MrSubidubi March 11, 2026 07:57
@davebalmer
Copy link
Author

Hopefully the changes work. Please let me know. This is my first time making an extension for Zed 😁

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