- 
                Notifications
    
You must be signed in to change notification settings  - Fork 279
 
chore: optimize assets #2513
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
base: main
Are you sure you want to change the base?
chore: optimize assets #2513
Conversation
Remove unused GIF and AVIF format configurations and simplify the sharp initialization by removing the animated flag logic. Add early logging of processed file paths.
          
WalkthroughThe  Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes The diff involves multiple distinct change types across a single file: removal of format support from type annotations and config objects, simplification of the image processing pipeline, reorganization of logging statements, addition of an explicit resize step, and modification of the async invocation pattern. While not overly complex, these heterogeneous changes require understanding the image optimization flow and verifying that the processing logic remains correct after the alterations. Pre-merge checks and finishing touches❌ Failed checks (1 warning, 1 inconclusive)
 ✅ Passed checks (1 passed)
 ✨ Finishing touches
 🧪 Generate unit tests (beta)
 📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (299)
 📒 Files selected for processing (1)
 ⏰ 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). (2)
 🔇 Additional comments (3)
 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. Comment   | 
    
What does this PR do?
(Provide a description of what this PR does.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)
Related PRs and Issues
(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)
Have you read the Contributing Guidelines on issues?
(Write your answer here.)
Summary by CodeRabbit