Skip to content

Compress logo#329

Merged
llrs-roche merged 1 commit intomainfrom
logo_size@main
Dec 10, 2024
Merged

Compress logo#329
llrs-roche merged 1 commit intomainfrom
logo_size@main

Conversation

@llrs-roche
Copy link
Contributor

It has been detected that the formatters logo is quite big in comparison to other packages.
This PR compress the logo so it no longer takes so much of the package size.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 10, 2024

✅ All contributors have signed the CLA
Posted by the CLA Assistant Lite bot.

@github-actions
Copy link
Contributor

Unit Tests Summary

  1 files    8 suites   15s ⏱️
 54 tests  54 ✅ 0 💤 0 ❌
372 runs  372 ✅ 0 💤 0 ❌

Results for commit c1e3803.

@github-actions
Copy link
Contributor

Unit Tests Summary

  1 files    8 suites   15s ⏱️
 54 tests  54 ✅ 0 💤 0 ❌
372 runs  372 ✅ 0 💤 0 ❌

Results for commit c1e3803.

@github-actions
Copy link
Contributor

badge

Code Coverage Summary

Filename             Stmts    Miss  Cover    Missing
-----------------  -------  ------  -------  -----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
R/format_value.R       197      12  93.91%   88, 105-112, 187, 204, 409, 423, 431
R/generics.R           125      12  90.40%   142, 280-284, 486, 498, 529, 559, 667, 680, 701-708
R/labels.R              55       7  87.27%   51, 57, 66, 107, 133, 142, 146
R/matrix_form.R        691      41  94.07%   131, 392, 512-513, 605, 615-618, 636, 667, 757-758, 772-777, 807-810, 870-871, 960-961, 988, 1016, 1068, 1230, 1266, 1269-1273, 1323, 1371, 1374, 1378
R/mpf_exporters.R      277      20  92.78%   2, 100-102, 147, 183, 227, 230, 235, 414-420, 424, 427, 431, 479, 557
R/page_size.R           42       1  97.62%   219
R/pagination.R         759      52  93.15%   327-330, 435-450, 540, 593, 598, 639, 677-688, 764, 876-877, 899-908, 1046, 1097-1098, 1249, 1286-1290, 1306, 1313, 1395, 1530-1531, 1547-1548, 1562-1563
R/tostring.R           783      66  91.57%   88, 296, 351, 421, 454, 462, 498, 555-558, 594, 658-661, 667-671, 674-677, 684-689, 772-773, 913-914, 979-986, 1036-1040, 1109, 1162, 1181-1185, 1196, 1214, 1231, 1246, 1344, 1385, 1430, 1516, 1555, 1609, 1616
R/utils.R                3       0  100.00%
R/zzz.R                 17       6  64.71%   28-33
TOTAL                 2949     217  92.64%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: c1e3803

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

@Melkiades Melkiades left a comment

Choose a reason for hiding this comment

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

Thanks! lgtm :)

@llrs-roche
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@llrs-roche llrs-roche merged commit 5ff3794 into main Dec 10, 2024
@llrs-roche llrs-roche deleted the logo_size@main branch December 10, 2024 14:59
@github-actions github-actions bot locked and limited conversation to collaborators Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants