Skip to content

fix: Make sure DefaultFormatRegionsProvider is the default IFormatRegionsProvider#1363

Merged
rubenporras merged 1 commit intoeclipse-lsp4e:mainfrom
FlorianKroiss:default-format-region-provider-ranking
Nov 6, 2025
Merged

fix: Make sure DefaultFormatRegionsProvider is the default IFormatRegionsProvider#1363
rubenporras merged 1 commit intoeclipse-lsp4e:mainfrom
FlorianKroiss:default-format-region-provider-ranking

Conversation

@FlorianKroiss
Copy link
Copy Markdown
Contributor

Comment thread org.eclipse.lsp4e.test/META-INF/MANIFEST.MF
Comment thread org.eclipse.lsp4e/src/org/eclipse/lsp4e/DocumentContentSynchronizer.java Outdated
@FlorianKroiss FlorianKroiss force-pushed the default-format-region-provider-ranking branch from d4b77ab to 445c62b Compare November 5, 2025 09:25
…ionsProvider

+ Add `service.ranking=1` to the properties of DefaultFormatRegionsProvider, so that it is properly used as the default
+ Extract lookup for IFormatRegionsProvider into Utility class
+ Add test cases for lookup
@rubenporras rubenporras force-pushed the default-format-region-provider-ranking branch from 445c62b to cafd204 Compare November 6, 2025 10:24
@rubenporras
Copy link
Copy Markdown
Contributor

@sebthom , the PR looks good to me. Since you participated in the Issue discussion, do you also agree we can merge this PR? If so I would merge it.

@sebthom
Copy link
Copy Markdown
Member

sebthom commented Nov 6, 2025

@rubenporras Fine with me

@rubenporras rubenporras merged commit 64f9ed6 into eclipse-lsp4e:main Nov 6, 2025
11 checks passed
@FlorianKroiss FlorianKroiss deleted the default-format-region-provider-ranking branch November 6, 2025 12:22
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.

IFormatRegionsProvider lookup does not fallback to default implementation

3 participants