anstyle icon indicating copy to clipboard operation
anstyle copied to clipboard

Improve internals to allow having much smaller HTML generated

Open GuillaumeGomez opened this issue 6 months ago • 3 comments

First commit comes from https://github.com/rust-cli/anstyle/pull/261.

For triagebot "enhanced" log pages, it makes a HUGE difference to not have all these tags rendered (takes me more than one minute to load a page with anstyle before this PR).

GuillaumeGomez avatar Jul 28 '25 15:07 GuillaumeGomez

Made atomic commits, moved function as suggested and created a new enum for span kinds (named SpanKind).

GuillaumeGomez avatar Jul 28 '25 22:07 GuillaumeGomez

Pull Request Test Coverage Report for Build 16598275897

Details

  • 43 of 46 (93.48%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.2%) to 53.717%

Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/anstyle-svg/src/lib.rs 43 46 93.48%
<!-- Total: 43 46
Files with Coverage Reduction New Missed Lines %
crates/anstyle/src/effect.rs 1 41.94%
<!-- Total: 1
Totals Coverage Status
Change from base Build 16581052420: 0.2%
Covered Lines: 1409
Relevant Lines: 2623

💛 - Coveralls

coveralls avatar Jul 28 '25 22:07 coveralls

Applied suggestions and merge tests updates within the relevant commits.

GuillaumeGomez avatar Jul 29 '25 12:07 GuillaumeGomez