Skip to content

docs: add rumdl to list of language servers for Markdown - #2842

Merged
rchl merged 1 commit into
sublimelsp:mainfrom
eread:eread/add-rumdl-to-list-of-language-servers-for-markdown
Mar 31, 2026
Merged

rchl merged 1 commit into
sublimelsp:mainfrom
eread:eread/add-rumdl-to-list-of-language-servers-for-markdown

Conversation

@eread

@eread eread commented Mar 29, 2026 •

Copy link
Copy Markdown
Contributor

Add rumdl to the list of language servers for Markdown: https://lsp.sublimetext.io/language_servers/#markdown.

For more information, see: https://rumdl.dev/lsp/.

Documentation preview: https://deploy-preview-2842--sublime-lsp.netlify.app/language_servers/#rumdl.

CC @rvben just FYI.

@netlify

netlify Bot commented Mar 29, 2026 •

Copy link
Copy Markdown

βœ… Deploy Preview for sublime-lsp ready!

Name Link
πŸ”¨ Latest commit 73f059c
πŸ” Latest deploy log https://app.netlify.com/projects/sublime-lsp/deploys/69cb1fdf67171d00072dfe94
😎 Deploy Preview https://deploy-preview-2842--sublime-lsp.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread docs/src/language_servers.md Resolved
@predragnikolic

Copy link
Copy Markdown
Member

To be consistent with other client configs:

https://lsp.sublimetext.io/language_servers/#assembly
https://lsp.sublimetext.io/language_servers/#clojure

I would suggest to inline the json configuration on the language_servers.md page.

Comment thread docs/src/language_servers.md Resolved
@rchl

rchl commented Mar 30, 2026

Copy link
Copy Markdown
Member

To be consistent with other client configs:

https://lsp.sublimetext.io/language_servers/#assembly https://lsp.sublimetext.io/language_servers/#clojure

I would suggest to inline the json configuration on the language_servers.md page.

I don't agree. If there is an upstream documentation the it's better to link to it to avoid having to maintain two places and potentially getting out of sync.

@jwortmann

Copy link
Copy Markdown
Member

I don't agree. If there is an upstream documentation the it's better to link to it to avoid having to maintain two places and potentially getting out of sync.

Ideally upstream docs would link to our docs page. Then we can easily update it if some parts of the configs change in the future, eg. point 2 in #2800.

To follow the conventions from the rest of the page, I'd say the server name should also not be formatted as inline code.

@eread
eread force-pushed the eread/add-rumdl-to-list-of-language-servers-for-markdown branch from b4e1910 to 73f059c Compare March 31, 2026 01:14
@eread

eread commented Mar 31, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback @predragnikolic @rchl @jwortmann

I've rebased and refactored this one. I'm on board with inlining the configuration here, and making that the source of truth. As @jwortmann said (#2842 (comment)), that would allow this project to document what is valid configuration.

Assuming this PR is merged, I'll raise a PR on the rumdl project to look at establishing the docs on this project as the source of truth.

@rchl
rchl merged commit e45b5ee into sublimelsp:main Mar 31, 2026
4 checks passed
@jfcherng

jfcherng commented Mar 31, 2026 •

Copy link
Copy Markdown
Contributor

Just a FYI. https://github.com/jfcherng-sublime/LSP-rumdl (it just downloads binary from GitHub)

I am fine to move it to sublimelsp org.

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.

5 participants