Skip to content

Conversation

@Geun-Oh
Copy link
Contributor

@Geun-Oh Geun-Oh commented Jan 27, 2025

Summary

Close: #4828

Previous PR: #4972

Add break if elem has multi attributes (at multiline side).

Test Plan

Modified test case for element with single attribute.

@github-actions github-actions bot added A-Project Area: project A-Formatter Area: formatter L-HTML Language: HTML labels Jan 27, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 27, 2025

CodSpeed Performance Report

Merging #4984 will not alter performance

Comparing Geun-Oh:multiline-attribute-position (ff7729f) with next (3538867)

Summary

✅ 95 untouched benchmarks

@dyc3 dyc3 merged commit d76e8d3 into biomejs:next Jan 28, 2025
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Formatter Area: formatter A-Project Area: project L-HTML Language: HTML

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants