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: Cleaned up whitespace processing and added whitespace tests #1267

Merged
merged 3 commits into from
May 16, 2023

Conversation

Sub6Resources
Copy link
Owner

Fixes #1146 and fixes #1251

@Sub6Resources Sub6Resources changed the title fix: Cleans up whitespace processing and adds whitespace tests fix: Cleaned up whitespace processing and added whitespace tests May 16, 2023
@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Patch coverage: 87.27% and project coverage change: +0.42 🎉

Comparison is base (fe896de) 59.31% compared to head (46fb455) 59.73%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1267      +/-   ##
==========================================
+ Coverage   59.31%   59.73%   +0.42%     
==========================================
  Files          37       37              
  Lines        2652     2690      +38     
==========================================
+ Hits         1573     1607      +34     
- Misses       1079     1083       +4     
Impacted Files Coverage Δ
lib/src/builtins/styled_element_builtin.dart 92.51% <37.50%> (-4.30%) ⬇️
lib/src/processing/whitespace.dart 85.47% <95.12%> (+0.58%) ⬆️
lib/src/builtins/text_builtin.dart 96.15% <100.00%> (+0.32%) ⬆️
lib/src/tree/replaced_element.dart 63.15% <100.00%> (+4.33%) ⬆️

... and 3 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Sub6Resources Sub6Resources merged commit cc00406 into master May 16, 2023
3 checks passed
@Sub6Resources Sub6Resources deleted the fix/whitespace-br branch May 16, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant