Skip to content

Deep clean#14

Merged
eldadfux merged 2 commits into
improve-screenshots-perffrom
deep-clean
Nov 20, 2025
Merged

Deep clean#14
eldadfux merged 2 commits into
improve-screenshots-perffrom
deep-clean

Conversation

@ItzNotABug

Copy link
Copy Markdown
Contributor

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

coderabbitai Bot commented Nov 17, 2025

Copy link
Copy Markdown

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

github-actions Bot commented Nov 17, 2025

Copy link
Copy Markdown

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
Copy Markdown
Contributor 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.

2 participants