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

npm ci および npm install が "install" コマンドとして扱われない #455

Open
nogic1008 opened this issue Apr 30, 2023 · 3 comments · May be fixed by #456
Open

npm ci および npm install が "install" コマンドとして扱われない #455

nogic1008 opened this issue Apr 30, 2023 · 3 comments · May be fixed by #456
Assignees

Comments

@nogic1008
Copy link

nogic1008 commented Apr 30, 2023

Describe the bug: バグの概要

npm ci コマンドnpm installコマンドと同じような動きをしますが、package-lock.jsonに基づいてより厳密にパッケージのインストールを行います。
そのため、GitHub ActionsのNode.jsテンプレートにもあるように、CI/CD環境でnpm installの代わりとして使われています。

しかしながら、このactionでは、BUILD_COMMANDnpm ciを含めても"install"コマンドとして扱われません。
そのため、不要であるはずのnpm install(およびnpm install --production)コマンドがビルド時に追加されてしまいます。

--ignore-scriptsを明示的に指定する必要がある場合1に、この挙動はビルドプロセスに影響を与えます。

To Reproduce: 再現手順

  1. 以下のようにnpm ciまたはnpm installを含むBUILD_COMMANDを指定する
steps:
  - uses: technote-space/release-github-actions@v8.0.3
    with:
      BUILD_COMMAND: npm ci --ignore-scripts && npm run build

Expected behavior: 期待する動作

  • npm ciBUILD_COMMANDコマンドに含む場合も、デフォルトのnpm installを実行しない

Screenshots: スクリーンショット

  • エラー発生時のログ
    • 余計なnpm install --productionコマンドが実行されてしまったことで、huskyコマンドがないエラーが発生している

Operating environment: バグが発生した環境

  • technote-space/release-github-actions@v8.0.3
  • ubuntu-latest hos runner

Footnotes

  1. huskyを利用している場合など

@welcome
Copy link

welcome bot commented Apr 30, 2023

🙌 Thanks for opening your first issue here! Be sure to follow the issue template!

@nogic1008
Copy link
Author

hasInstallCommandの条件式が原因だと思われますが、現状では仮にnpm installを指定しても"install"コマンドとして扱われません。
(npmの場合はnpm run installと指定しないとhasInstallCommandtrueにならない)

@nogic1008 nogic1008 changed the title npm ci が "install" コマンドとして扱われない npm ci および npm install が "install" コマンドとして扱われない Apr 30, 2023
@nogic1008 nogic1008 linked a pull request Apr 30, 2023 that will close this issue
@nogic1008
Copy link
Author

@technote-space Any updates?

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 a pull request may close this issue.

2 participants