ci: skip publish jobs when version already exists in registry#15
Conversation
…rm decoding JPEG XL now supports both encoding and decoding via libjxl. AVIF decoding is available on all platforms via statically linked dav1d. Added jxl to the convert format list in usage docs.
Add a check-versions job that queries crates.io, Maven Central, and PyPI APIs before running build/publish jobs. When a version is already published, the entire job chain is skipped via dependency propagation, saving CI time and costs.
Summary of ChangesHello @clroot, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request primarily updates documentation files to accurately reflect the current state of image format support within the project. Specifically, it highlights the addition of JPEG XL encoding capabilities and provides more detailed instructions for AVIF decoder setup. It is important to note that the provided patches do not align with the pull request title and description, which refer to CI skip jobs; the changes are exclusively documentation-related. Highlights
Changelog
Ignored Files
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request appears to add support for JPEG XL encoding and updates the build process for the AVIF decoder, including documentation changes. However, the pull request title ('ci: skip publish jobs when version already exists in registry') and description are about CI optimizations, which seems to be unrelated to the file changes. To avoid confusion for reviewers and to maintain a clear commit history, it is highly recommended to update the PR title and description to accurately reflect the addition of JPEG XL support.
| - nasm (MozJPEG / rav1e 어셈블리 최적화용) | ||
| - dav1d (macOS만, AVIF 디코딩용) | ||
| - meson + ninja (dav1d AVIF 디코더 소스 빌드용) | ||
| - `SYSTEM_DEPS_DAV1D_BUILD_INTERNAL=always` 설정으로 dav1d를 소스에서 빌드 |
There was a problem hiding this comment.
This list item reads like an instruction. To maintain consistency with the other requirements in the list (which are nouns like 'C compiler'), it would be clearer to phrase this as a requirement as well.
| - `SYSTEM_DEPS_DAV1D_BUILD_INTERNAL=always` 설정으로 dav1d를 소스에서 빌드 | |
| - `SYSTEM_DEPS_DAV1D_BUILD_INTERNAL=always` 환경 변수 (dav1d 소스 빌드용) |
Summary
check-versionsjob that queries crates.io, Maven Central, and PyPI APIs to detect already-published versionsifconditions and dependency propagation when version existsTest plan
check-versionsjob correctly detects existing versions (e.g., v0.5.0)🤖 Generated with Claude Code