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

docker compose buildの失敗するのを修正 #821

Merged
merged 7 commits into from
Dec 30, 2024

Conversation

yonta
Copy link
Collaborator

@yonta yonta commented Dec 30, 2024

fix #821

やったこと

  • dev.Dockerfileで使うlibpq-devのバージョンを正しく指定した
  • DockerfileのSHELLオプションに-eを追加し、heredoc内スクリプト失敗時にエラー停止するようにした
  • dev.Dockerfileのsyntaxバージョン指定を1.4 -> 1に変更
    • 昔はheredocを使うために1.4指定が必要だったが、今は必要ない
  • dev.Dockerfileにおけるpostgresql/yarn/nodeのインストールを分割した
    • 今まではインストールエラー時にどのインストールでエラーしたかわからなかった
    • どれがエラーしたかわかりやすい!
  • dev.Dockerfileにおけるgem installの2行を1行にまとめた
  • docker build checkの修正に伴い、docker compose buildをオプションなしで動くようにした

heredocなどでスクリプト失敗を無視せず、失敗でエラー終了するようにする。
heredocを使うために1.4指定だったが、1でも使えるようになっていた。
今までは同じRUNで行っていた。
どれかがエラーするとどれでエラーしたかわからなかった。
これからは分離したため、どのインストールが失敗したかわかりやすい。

また、キャッシュ削除は最後に行うため、hadolintを一部ignoreしている。
Copy link

github-actions bot commented Dec 30, 2024

📦 esbuild Bundle Analysis for sakazuki

This analysis was generated by esbuild-bundle-analyzer. 🤖
This PR introduced no changes to the esbuild bundle! 🙌

@momocus momocus merged commit d4958ff into main Dec 30, 2024
5 checks passed
@momocus momocus deleted the pr/issue820/fix-docker-build branch December 30, 2024 13:17
@momocus momocus restored the pr/issue820/fix-docker-build branch December 30, 2024 13:18
@yonta yonta deleted the pr/issue820/fix-docker-build branch December 30, 2024 13:19
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.

2 participants