Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: use vscode-langservers-extracted instead of vscode-css-langageserver-bin #741

Merged
merged 2 commits into from
May 27, 2024

Conversation

ver-1000000
Copy link
Contributor

closed #740 .

  • before:
    • 20240501165227
  • after:
    • 20240501165200

git grep vscode-css-languageserver-binして、関連していそうなコードを削除し、次のコードを該当箇所にコピーしました。

他に考慮すべきところや問題点があればお知らせいただければと思います🙏

@mattn
Copy link
Owner

mattn commented May 1, 2024

ありがとうございます。エントリを消してしまうとアンインストールできなくなるので、いったん deprecated 表示の仕組みを検討したいと思います。

@ver-1000000
Copy link
Contributor Author

エントリを消してしまうとアンインストールできなくなるので

おお、そうなのですね……! 私の方で必要な然るべき対応があればお申し付けください。

(場合によってはよしなにクローズしちゃってください🙏)

@mattn
Copy link
Owner

mattn commented May 1, 2024

ありがとうございます。deprecated 表示を入れたんで、settings.json で css-languageserver を消したり上書きしたりせず、css-languageserver よりも前に(配列の1つ前)、vscode-css-language-server が来るように変更頂けますか。(bin を消す修正は取り込みたいです)

@ver-1000000
Copy link
Contributor Author

わざわざありがとうございます…… 対応したのでご確認いただければと思います!

(Web上でSyncボタン押したら意図しない変なMergeコミットが入ってしまったので、upstreamから再度masterを切ってcherry-pick後、コミット作ってforce-pushしています)

@mattn mattn merged commit 9747290 into mattn:master May 27, 2024
14 checks passed
@mattn
Copy link
Owner

mattn commented May 27, 2024

ありがとうございます。

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.

vscode-css-languageserver-bin is too outdated.
2 participants