Skip to content

fix: 统一环境变量为 IMAGE_COMPRESS_API_ 前缀#8

Merged
zzgosh merged 1 commit intomainfrom
codex/prefix-env-vars
Mar 8, 2026
Merged

fix: 统一环境变量为 IMAGE_COMPRESS_API_ 前缀#8
zzgosh merged 1 commit intomainfrom
codex/prefix-env-vars

Conversation

@zzgosh
Copy link
Copy Markdown
Owner

@zzgosh zzgosh commented Mar 8, 2026

变更说明

  • 将服务使用的环境变量统一改为 IMAGE_COMPRESS_API_ 前缀,避免与同机其他服务冲突。
  • 启动时显式拒绝旧的未加前缀变量名,并给出对应的新变量名提示。
  • 同步更新 .env.exampleREADME.mdDockerfile 与相关测试。

验证

  • npm run check
  • npm test
  • npm run build

@zzgosh
Copy link
Copy Markdown
Owner Author

zzgosh commented Mar 8, 2026

@codex review code in Chinese language

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f44e63bbdb

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@zzgosh zzgosh merged commit 2a0c742 into main Mar 8, 2026
1 check passed
@zzgosh zzgosh deleted the codex/prefix-env-vars branch March 8, 2026 13:47
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