Skip to content

Preserve trailing statement semicolons when using fmt: skip#8273

Merged
MichaReiser merged 1 commit intomainfrom
fmt-skip-preserve-semicolon
Oct 30, 2023
Merged

Preserve trailing statement semicolons when using fmt: skip#8273
MichaReiser merged 1 commit intomainfrom
fmt-skip-preserve-semicolon

Conversation

@MichaReiser
Copy link
Member

@MichaReiser MichaReiser commented Oct 27, 2023

Summary

Fixes #8263

We have the same problem with fmt: off. I'll look into this separately

Test Plan

Added tests, verified that the similarity index is unchanged

@MichaReiser MichaReiser added the formatter Related to the formatter label Oct 27, 2023
@MichaReiser
Copy link
Member Author

MichaReiser commented Oct 27, 2023

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2023

PR Check Results

Ecosystem

✅ ecosystem check detected no linter changes.

✅ ecosystem check detected no format changes.

@MichaReiser MichaReiser force-pushed the fmt-skip-preserve-semicolon branch from 538867c to 44d4cac Compare October 29, 2023 23:41
@codspeed-hq
Copy link

codspeed-hq bot commented Oct 29, 2023

CodSpeed Performance Report

Merging #8273 will not alter performance

Comparing fmt-skip-preserve-semicolon (44d4cac) with main (e799f90)

Summary

✅ 25 untouched benchmarks

@MichaReiser MichaReiser merged commit 2c84f91 into main Oct 30, 2023
@MichaReiser MichaReiser deleted the fmt-skip-preserve-semicolon branch October 30, 2023 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

formatter Related to the formatter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

# fmt: skip strips trailing semicolon

2 participants