Skip to content

feat: add analysis for spacing resets #467

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

Merged
merged 1 commit into from
Jun 13, 2025
Merged

feat: add analysis for spacing resets #467

merged 1 commit into from
Jun 13, 2025

Conversation

bartveneman
Copy link
Member

Prior art: csssstats.com

This analysis checks for:

  • scientific notation (0e0)
  • signed numbers (+0, -0)
  • floats (0.0)
  • with or without units (0, 0em)
  • vendor prefixed property names (-webkit-padding-inline)
  • all of the values in a shorthand being zero (0 0 0 0, 0px 0 0em)
  • combinations of the above (-moz-margin: 0 0px +0.0e0em)

Implementation sparked by @ohhelloana

@bartveneman bartveneman added the ✨ enhancement New feature or request label Jun 13, 2025
@codecov-commenter
Copy link

Bundle Report

Changes will increase total bundle size by 728 bytes (0.26%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
analyzeCss-esm 30.82kB 728 bytes (2.42%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: analyzeCss-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
css-*.js 728 bytes 30.82kB 2.42%

Files in css-*.js:

  • ./src/values/values.js → Total Size: 971 bytes

  • ./src/index.js → Total Size: 31.1kB

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.87%. Comparing base (704d7ac) to head (3d98b8a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #467   +/-   ##
=======================================
  Coverage   99.86%   99.87%           
=======================================
  Files          18       18           
  Lines        2262     2320   +58     
=======================================
+ Hits         2259     2317   +58     
  Misses          3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bartveneman bartveneman merged commit 36cb587 into main Jun 13, 2025
4 checks passed
@bartveneman bartveneman deleted the resets branch June 13, 2025 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants