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

chore: use alpine linux to minimize image size #248

Draft
wants to merge 1 commit into
base: sleepyfox
Choose a base branch
from

Conversation

Integral-Tech
Copy link
Contributor

@Integral-Tech Integral-Tech commented Jan 1, 2025

Summary by Sourcery

使用 Alpine Linux 基础镜像以减小 Docker 镜像大小。

构建:

  • 使用 node:22-alpine 作为基础镜像。
  • 使用 alpine:latest 作为 tgs-to-gif 构建镜像。
Original summary in English

Summary by Sourcery

Use alpine linux base image to reduce the docker image size.

Build:

  • Use node:22-alpine as base image.
  • Use alpine:latest as tgs-to-gif-build image.

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我们已跳过审查此拉取请求。我们不审查打包更改 - 如果您希望我们更改此设置,请联系我们

Original comment in English

We have skipped reviewing this pull request. We don't review packaging changes - Let us know if you'd like us to change this.

Copy link

sourcery-ai bot commented Jan 1, 2025

Here's the translation to Chinese:

审阅者指南 by Sourcery

此 PR 更改了 Dockerfile 中使用的基础镜像,以减小整体镜像大小。node 镜像从 node:22-slim 更改为 node:22-alpinetgs-to-gif-build 镜像从 debian:bookworm-slim 更改为 alpine:latest

由于更改看起来简单,未生成任何图表。

文件级更改

更改 详情 文件
使用 alpine Linux 作为基础镜像
  • node 基础镜像从 node:22-slim 更改为 node:22-alpine
  • tgs-to-gif-build 基础镜像从 debian:bookworm-slim 更改为 alpine:latest
Dockerfile

提示和命令

与 Sourcery 交互

  • 触发新审阅: 在拉取请求中评论 @sourcery-ai review
  • 继续讨论: 直接回复 Sourcery 的审阅评论。
  • 从审阅评论生成 GitHub 问题: 通过回复评论要求 Sourcery 创建问题。
  • 生成拉取请求标题: 在拉取请求标题的任何位置写 @sourcery-ai 以随时生成标题。
  • 生成拉取请求摘要: 在拉取请求正文的任何位置写 @sourcery-ai summary 以随时生成 PR 摘要。您还可以使用此命令指定摘要的插入位置。

自定义您的体验

访问您的仪表板以:

  • 启用或禁用审阅功能,如 Sourcery 生成的拉取请求摘要、审阅者指南等。
  • 更改审阅语言。
  • 添加、删除或编辑自定义审阅说明。
  • 调整其他审阅设置。

获取帮助

Original review guide in English

Reviewer's Guide by Sourcery

This PR changes the base images used in the Dockerfile to reduce the overall image size. The node image is changed from node:22-slim to node:22-alpine, and the tgs-to-gif-build image is changed from debian:bookworm-slim to alpine:latest.

No diagrams generated as the changes look simple and do not need a visual representation.

File-Level Changes

Change Details Files
Use alpine Linux for base images
  • Change node base image from node:22-slim to node:22-alpine
  • Change tgs-to-gif-build base image from debian:bookworm-slim to alpine:latest
Dockerfile

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time. You can also use
    this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@Integral-Tech
Copy link
Contributor Author

As gosu, rlottie and rlottie-dev are currently only available in testing repository of Alpine Linux, this pull request cannot proceed now. Using testing repository in Alpine Linux, especially in release branch, is not recommended in production.

@Integral-Tech Integral-Tech marked this pull request as draft January 1, 2025 17:00
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