Skip to content

Conversation

RubenSandwich
Copy link
Contributor

@RubenSandwich RubenSandwich commented Oct 2, 2025

The initial commit is limited to TFE v1.0.x, as lots of images are replicated across all TFE versions. Leading to ~7.5k images total. So by limiting to just TFE v1.0.x we only have 136 to compare.

Care has been taken to make sure that no noticeable differences have been introduced through compression.

Once the visual compare has passed education the rest of the TFE images will be compressed as well.

Copy link
Contributor

github-actions bot commented Oct 2, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Tue Oct 7 14:08:04 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Tue Oct 7 13:59:15 UTC 2025

Copy link
Contributor

github-actions bot commented Oct 2, 2025

Broken Link Checker

No broken links found! 🎉

@RubenSandwich RubenSandwich marked this pull request as ready for review October 7, 2025 13:03
@RubenSandwich RubenSandwich requested review from a team as code owners October 7, 2025 13:03
Copy link
Contributor

@Younique98 Younique98 left a comment

Choose a reason for hiding this comment

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

Lgtm!!

Copy link
Contributor

@nikolasrieble nikolasrieble left a comment

Choose a reason for hiding this comment

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

Thank you very much!

I reviewed a few samples, and see no issues. Obviously I can not review all images.

@RubenSandwich RubenSandwich merged commit 1400662 into main Oct 7, 2025
21 checks passed
@RubenSandwich RubenSandwich deleted the rn/tfe-image-compression-oct-2 branch October 7, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants