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

uroborosql-fmtが変更されるとnpm installが失敗するようになる #25

Open
ppputtyo opened this issue Oct 25, 2023 · 1 comment

Comments

@ppputtyo
Copy link
Contributor

uroborosql-fmtのmainブランチが変更されるとnpm installが失敗するようになります。
原因を調査し、とりあえず思いついた解決案を2つ記載しているので意見をいただけると幸いです。

再現方法

cloneしていない場合

  1. git clone
  2. npm install

既にcloneしている場合

  1. server/以下のnode_module削除
  2. npm install

原因

  1. uroborosql-fmtのmainブランチの変更によりnapiがアップデートされる
  2. そのnapiを持ってきてnpm installするとserver/package-lock.jsonのバージョンと違うのでエラー

解決案

なんらかの方法でpackage-lock.jsonuroborosql-fmt-napi-0.0.0.tgzのバージョンを合わせたい

1. uroborosql-fmtの変更と同時にvscode-uroborosql-fmtの方のアップデートも行う

package-lock.jsonを常に最新にして問題を解決する

メリット

  • 単純

デメリット

  • 手間がかかる
  • 拡張機能に関係のない変更の場合でもアップデートの必要がある
    • これは、uroborosql-fmtの方で拡張機能に関係のある変更の場合のみuroborosql-fmt-napi-0.0.0.tgzを再生成するようにすることで解決しそう

2. uroborosql-fmtのgh-pagesブランチをバージョン分けする

古いバージョンのuroborosql-fmt-napi-0.0.0.tgzもダウンロード可能にすることで解決する

メリット

  • uroborosql-fmtの変更があった場合に、vscode-uroborosql-fmtを変更しなくてもnpm installが成功する

デメリット

  • バージョン管理の手間がかかる
    • 過去のバージョンを全て残す、または必要なバージョン以外は削除する、など運用方法も決めなければいけない
@ota-meshi
Copy link
Member

#26 でとりあえず、エラーにならないように修正してみました。
ただし、uroborosql-fmt-napiが更新された場合、npm installでpackage-lock.json が更新されてしまいます。
そのため、これは一時的な回避策であると思っています。

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

No branches or pull requests

2 participants