Skip to content

Conversation

@ItzNotABug
Copy link
Member

What does this PR do?

Excessively remove libs after a thorough test and dirs navigation!

Test Plan

Manual. Unit and E2Es.

Related PRs and Issues

Have you read the Contributing Guidelines on issues?

Yes.

@ItzNotABug ItzNotABug self-assigned this Nov 17, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 17, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch deep-clean

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ItzNotABug ItzNotABug changed the base branch from main to improve-screenshots-perf November 17, 2025 06:06
@github-actions
Copy link

github-actions bot commented Nov 17, 2025

Docker Image Stats

Metric Value
Image Size 439MB
Memory Usage 156.7MiB
Cold Start Time 1.01s
Screenshot Time 3.58s

Screenshot benchmark: Average of 3 runs on https://appwrite.io

@ItzNotABug ItzNotABug marked this pull request as ready for review November 17, 2025 06:12
@ItzNotABug ItzNotABug requested a review from Meldiron November 17, 2025 06:59
@ItzNotABug
Copy link
Member Author

Tested this variant locally with Appwrite avatars, e2e tests and sites, worked OK.

@eldadfux eldadfux merged commit 6d0b343 into improve-screenshots-perf Nov 20, 2025
4 checks passed
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.

3 participants