Skip to content

Dockerfile production stageにbuild-essentialを追加#9349

Merged
komagata merged 1 commit intomainfrom
fix/dockerfile-add-build-essential
Dec 9, 2025
Merged

Dockerfile production stageにbuild-essentialを追加#9349
komagata merged 1 commit intomainfrom
fix/dockerfile-add-build-essential

Conversation

@komagata
Copy link
Copy Markdown
Member

@komagata komagata commented Dec 9, 2025

Summary

  • bigdecimal等のネイティブ拡張gemのビルドに必要なコンパイラがproduction stageに不足していたため追加

Test plan

  • Dockerビルドが成功することを確認

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

  • 変更
    • 本番環境のイメージに、追加のツールおよびライブラリをインストールするよう更新しました。これには build-essential、curl、gnupg2、git、ca-certificates、libvips が含まれます。

✏️ Tip: You can customize this high-level summary in your review settings.

bigdecimal等のネイティブ拡張gemのビルドに必要なコンパイラが
production stageに不足していたため追加。

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 9, 2025

Walkthrough

本番ステージのDockerfileにおいて、インストール対象パッケージが拡張されました。「ランタイムのみの依存」から「ランタイムおよびビルド依存」へコメントが変更され、build-essential、curl、gnupg2、git、ca-certificates、libvipsなどのツールとライブラリが追加されています。

Changes

コホート / ファイル 変更内容
本番ステージの依存パッケージ拡張
Dockerfile
本番ステージのapt-get installコマンドにビルド関連パッケージ(build-essential、curl、gnupg2、git、ca-certificates、libvips)を追加。コメント文字列も「Install only runtime dependencies」から「Install runtime and build dependencies」に更新。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • インストール対象パッケージの一覧追加に伴う意図・用途の確認
  • 本番イメージサイズの増加影響の検討
  • 追加されたパッケージ各々の必要性の妥当性確認

Poem

🐰 ビルドツール増えて、
本番も充実、
依存もクリア、
完成へ進む!

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive 説明は変更の概要とテストプランを含めていますが、リポジトリの指定テンプレートの必須セクション(Issue番号、変更確認方法の詳細、Screenshotなど)が不足しており、テンプレートとの整合性が欠けている。 指定されたテンプレートに従い、Issue番号、詳細な変更確認方法、変更前後のスクリーンショット(またはコマンド出力)を追加してください。
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed タイトルはDockerfileのproduction stageにbuild-essentialを追加する主要な変更を明確に要約しており、リポジトリの主な目的と一致している。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/dockerfile-add-build-essential

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 724c074 and b80f4e5.

📒 Files selected for processing (1)
  • Dockerfile (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📚 Learning: 2025-11-26T08:23:30.927Z
Learnt from: CR
Repo: fjordllc/bootcamp PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-26T08:23:30.927Z
Learning: Use `bin/setup` for local database setup and dependency installation; avoid manual tweaks in `config/` without team discussion

Applied to files:

  • Dockerfile
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build_and_test

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@komagata komagata merged commit 12bca84 into main Dec 9, 2025
4 checks passed
@komagata komagata deleted the fix/dockerfile-add-build-essential branch December 9, 2025 19:46
@github-actions github-actions bot mentioned this pull request Dec 9, 2025
28 tasks
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.

1 participant