Skip to content
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

fix(noUnknownProperty): prevent warning on CSS custom property #3193

Merged
merged 3 commits into from
Jun 13, 2024

Conversation

chansuke
Copy link
Member

Summary

Closes #3092.

Prevent warnings from being thrown for Custom properties (--*).

Test Plan

@chansuke chansuke changed the title fix:(noUnknownProperty): prevent warning on CSS custom property. fix:(noUnknownProperty): prevent warning on CSS custom property Jun 12, 2024
@github-actions github-actions bot added L-CSS Language: CSS A-Changelog Area: changelog labels Jun 12, 2024
Copy link

codspeed-hq bot commented Jun 12, 2024

CodSpeed Performance Report

Merging #3193 will improve performances by ×2.6

Comparing chansuke:fix/no-unknown-property (a0af0e2) with main (a4b050b)

Summary

⚡ 2 improvements
✅ 88 untouched benchmarks

Benchmarks breakdown

Benchmark main chansuke:fix/no-unknown-property Change
css_analyzer[bootstrap.css] 226.9 ms 89 ms ×2.6
css_analyzer[bulma.css] 92.4 ms 77.7 ms +18.82%

@Sec-ant
Copy link
Member

Sec-ant commented Jun 12, 2024

There must be so many custom properties in bootstrap.css given the performance numbers. 😆

@chansuke chansuke changed the title fix:(noUnknownProperty): prevent warning on CSS custom property fix(noUnknownProperty): prevent warning on CSS custom property Jun 13, 2024
@chansuke chansuke requested a review from Sec-ant June 13, 2024 03:28
@Sec-ant Sec-ant merged commit 2e06a17 into biomejs:main Jun 13, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Changelog Area: changelog L-CSS Language: CSS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

💅 noUnknownProperty throws warning for CSS custom properties
2 participants