Skip to content

chore: remove src from published package files#27

Merged
RostiMelk merged 2 commits intomainfrom
chore/remove-src-from-package-files
Feb 9, 2026
Merged

chore: remove src from published package files#27
RostiMelk merged 2 commits intomainfrom
chore/remove-src-from-package-files

Conversation

@RostiMelk
Copy link
Member

All entry points resolve to dist/. The src directory is unused by consumers.

@vercel
Copy link

vercel bot commented Feb 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
react-logo-soup Ready Ready Preview, Comment Feb 9, 2026 8:11pm

Request Review

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

Benchmark Comparison: main vs chore/remove-src-from-package-files

Threshold: 5%+ change, >100us absolute delta, and statistically significant (p<0.05).

Benchmark main chore/remove-src-from-package-files Change p-value Verdict
content detection (1 logo) 33.13 us 32.04 us -3.3% 0.621 unchanged
render pass (20 logos) 1.11 us 1.10 us -1.5% 0.530 unchanged
mount 20 logos (no detection) 3.20 us 3.49 us +8.9% 0.306 unchanged
mount 20 logos (defaults) 759.41 us 753.48 us -0.8% 0.522 unchanged

No regressions detected.

Feature cost breakdown

How expensive are individual features? Measured on this run's HEAD commit.

Feature On Off Cost Sig
densityAware: true vs false 30.29 us 30.13 us ~same 0.860
alignBy: visual-center-y vs bounds 2.12 us 69 ns ~same 0.057
cropToContent: true vs false 2.19 ms 54 ns 2.19 ms <0.001 ***
layout update: full mount vs cached 755.47 us 2.21 us 342× slower <0.001 ***

Full benchmark output in the CI job logs.

@RostiMelk RostiMelk merged commit 101faeb into main Feb 9, 2026
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.

1 participant