atsushieno / vscode-language-review

Re:VIEW language Support for Visual Studio Code. / issue/PRは日本語でも対応できます
Other
43 stars 8 forks source link

review.js-vscodeのバージョンアップ #35

Closed TM0428 closed 1 week ago

TM0428 commented 3 weeks ago

本家のnpmバージョンが更新されていないため、 @<del>{}等のinlineが機能していませんでしたが、バージョンアップしたパッケージを利用することで使えるようになっています。

atsushieno commented 3 weeks ago

PRありがとうございます。 このforkの作者の方は存じ上げないので、一応意図しない危ない変更がないか差分を確認してみましたが、この中にinlineまわりに該当する変更があるということでしょうか? https://github.com/yfakariya/review.js-vscode/compare/master...bufferings:review.js-vscode:master

新しいforkは fe438d0 の変更(エラーチェックのコメントアウトや無関係なファイルの削除など)がいささか危ういように見えたので、本家に反映できる変更というより個人的に使っている修正版という印象が強く、現状こちらのforkをこちらのvscode拡張で正として参照することにはやや抵抗感があります。もし本家review.js-vscodeのパッケージが0.19.2以降が不在であることが問題であるとしたら、パッケージ発行のリクエストを出せば解決する問題かもしれません。

TM0428 commented 3 weeks ago

もし本家review.js-vscodeのパッケージが0.19.2以降が不在であることが問題であるとしたら、パッケージ発行のリクエストを出せば解決する問題かもしれません。

そうですね。現在配布されているバージョンが0.19.2でeef48edcfb3ba60ca58fcb2e535bfb3a96be64c4を使用していますが、最新のmasterが反映されたバージョン(0.19.3?)が配布されていないのが原因です。

一度review.js-vscodeのIssueに投げて対応してもらえるか相談してからPR出しますね。

atsushieno commented 1 week ago

こちら、手元で最新版にアップデートできることを確認してありますが、こちらで更新してしまいましょうか? (その場合PRはunmerged closedになります)

TM0428 commented 1 week ago

こちら、公式のバージョンアップのコードに反映完了しましたので、再レビューお願いします

atsushieno commented 1 week ago

ありがとうございます!

atsushieno commented 1 week ago

少し依存パッケージの変更を加えて、open-vsxとvscode marketplaceの両方にアップデートを発行しました。もしうまく動作しないことがあったらお知らせください。