Skip to content

feat(oxlint): add language server, bump to 0.15.8#8717

Merged
Conarius merged 3 commits intomason-org:mainfrom
marekvospel:patch-1
Mar 26, 2025
Merged

feat(oxlint): add language server, bump to 0.15.8#8717
Conarius merged 3 commits intomason-org:mainfrom
marekvospel:patch-1

Conversation

@marekvospel
Copy link
Contributor

Describe your changes

This merge request updates the oxlint package to use npm. (Sorry for not waiting for a response in the related issue, I had too much free time on my hands 😂) It also adds the oxlint_language_server binary.

Issue ticket number and link

Closes #8709

Checklist before requesting a review

  • I have successfully tested installation of the package.
  • I have successfully tested the package after installation.

Screenshots

Installed
lsp setup

Copy link
Member

@mehalter mehalter left a comment

Choose a reason for hiding this comment

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

Sounds good! If they eventually release binaries with github releases for both concurrently then we might want to look into separating them out then. That way we also don't have to rely on user npm dependency. One small chance requested now to add the correct vscode schema

Co-authored-by: Micah Halter <micah@mehalter.com>
Copy link
Contributor

@Conarius Conarius left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@mehalter mehalter left a comment

Choose a reason for hiding this comment

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

Looks good to me as long as tests pass!

@Conarius Conarius merged commit 1d77b67 into mason-org:main Mar 26, 2025
18 checks passed
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.

Old oxlint version, homepage, missing oxc_language_server

4 participants