r-sugi / nextjs-tdd-template

https://nextjs-tdd-templatestorybook-rsugis-projects.vercel.app
0 stars 0 forks source link

fix: image of debian:bullseye-slim #208

Closed r-sugi closed 2 months ago

r-sugi commented 2 months ago

User description

207


PR Type

enhancement


Description


Changes walkthrough 📝

Relevant files
Enhancement
Dockerfile
Dockerfileのベースイメージとインストール方法の最適化                                                   

functions/Dockerfile
  • ベースイメージをubuntu:22.04からdebian:bullseye-slimに変更
  • 不要なパッケージのインストールを避けるため--no-install-recommendsオプションを追加
  • apt-get cleanrm -rf /var/lib/apt/lists/*でイメージサイズを最適化
  • +5/-3     

    💡 PR-Agent usage: Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    vercel[bot] commented 2 months ago

    The latest updates on your projects. Learn more about Vercel for Git ↗︎

    Name Status Preview Comments Updated (UTC)
    nextjs-tdd-template_storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 15, 2024 3:50am
    github-actions[bot] commented 2 months ago

    PR Reviewer Guide 🔍

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Key issues to review

    Syntax Error
    6行目の`apt install -y --no-install-recommends nodejs -`には不要なハイフンが含まれています。これは構文エラーを引き起こす可能性があります。
    github-actions[bot] commented 2 months ago

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Possible bug
    不要なハイフンを削除する ___ **`nodejs` のインストールコマンドに余分なハイフンが含まれています。これを削除してください。** [functions/Dockerfile [6]](https://github.com/r-sugi/nextjs-tdd-template/pull/208/files#diff-88224d978d16d31ce1a8cc4ceeb3aa8e4629df630b8821992f2e5b64f81b6c0bR6-R6) ```diff -apt install -y --no-install-recommends nodejs - +apt install -y --no-install-recommends nodejs ```
    Suggestion importance[1-10]: 10 Why: Removing the unnecessary hyphen from the `nodejs` installation command fixes a potential bug, ensuring the command executes correctly.
    10
    Best practice
    apt installapt-get install に変更してコマンドの一貫性を保つ ___ **apt install コマンドを apt-get install に統一してください。これにより、コマンドの一貫性が保たれ、予期しない挙動を避けることができます。** [functions/Dockerfile [4-6]](https://github.com/r-sugi/nextjs-tdd-template/pull/208/files#diff-88224d978d16d31ce1a8cc4ceeb3aa8e4629df630b8821992f2e5b64f81b6c0bR4-R6) ```diff -apt install -y --no-install-recommends curl openjdk-17-jdk && -apt install -y --no-install-recommends nodejs - +apt-get install -y --no-install-recommends curl openjdk-17-jdk && +apt-get install -y --no-install-recommends nodejs - ```
    Suggestion importance[1-10]: 9 Why: Changing `apt install` to `apt-get install` improves consistency and reduces the risk of unexpected behavior, which is a best practice in Dockerfiles.
    9
    Enhancement
    apt-get cleanrm -rf /var/lib/apt/lists/* を組み合わせてイメージサイズを削減する ___ **apt-get cleanrm -rf /var/lib/apt/lists/*
    を組み合わせることで、不要なファイルを効果的に削除し、イメージサイズを最小限に抑えます。このコマンドを RUN コマンドとして一つにまとめることを提案します。** [functions/Dockerfile [7-8]](https://github.com/r-sugi/nextjs-tdd-template/pull/208/files#diff-88224d978d16d31ce1a8cc4ceeb3aa8e4629df630b8821992f2e5b64f81b6c0bR7-R8) ```diff -apt-get clean && -rm -rf /var/lib/apt/lists/* +RUN apt-get clean && \ + rm -rf /var/lib/apt/lists/* ```
    Suggestion importance[1-10]: 8 Why: Combining these commands into a single `RUN` command is a good enhancement that helps reduce the image size by effectively cleaning up unnecessary files.
    8
    github-actions[bot] commented 2 months ago

    Preview APP: 'https://nextjs-tdd-template-jqtsuxb55-rsugis-projects.vercel.app'

    github-actions[bot] commented 2 months ago

    Preview APP: 'https://nextjs-tdd-template-bagt2yxvb-rsugis-projects.vercel.app'