Skip to content

Fix legacy_swiftui_aspect_ratio not preserving whitespace (#6883) - #6885

Merged
SimplyDanny merged 2 commits into
realm:mainfrom
CraigSiemens:fix-legacy-swiftui-aspect-ratio-lost-whitespace
Aug 24, 2026
Merged

Fix legacy_swiftui_aspect_ratio not preserving whitespace (#6883)#6885
SimplyDanny merged 2 commits into
realm:mainfrom
CraigSiemens:fix-legacy-swiftui-aspect-ratio-lost-whitespace

Conversation

@CraigSiemens

Copy link
Copy Markdown
Contributor

Fixes #6883 by editing the baseName of the MemberAccessExprSyntax to preserve the existing trivia (whitespace) around the function call.

Previously it was creating a new expression for the function call so the result would always be on one line.

@SwiftLintBot

SwiftLintBot commented Aug 22, 2026

Copy link
Copy Markdown
20 Messages
📖 Building this branch resulted in a binary size of 28460.6 KiB vs 28460.51 KiB when built on main (0% larger).
📖 Linting Aerial with this PR took 0.1 s vs 0.11 s on main (9% faster).
📖 Linting Alamofire with this PR took 0.12 s vs 0.13 s on main (7% faster).
📖 Linting Brave with this PR took 0.41 s vs 0.4 s on main (2% slower).
📖 Linting Brigade with this PR took 0.98 s vs 0.99 s on main (1% faster).
📖 Linting DuckDuckGo with this PR took 1.96 s vs 1.96 s on main (0% slower).
📖 Linting Firefox with this PR took 0.76 s vs 0.73 s on main (4% slower).
📖 Linting Kickstarter with this PR took 0.48 s vs 0.46 s on main (4% slower).
📖 Linting Moya with this PR took 0.11 s vs 0.11 s on main (0% slower).
📖 Linting NetNewsWire with this PR took 0.2 s vs 0.19 s on main (5% slower).
📖 Linting Nimble with this PR took 0.11 s vs 0.13 s on main (15% faster).
📖 Linting PocketCasts with this PR took 0.5 s vs 0.53 s on main (5% faster).
📖 Linting Quick with this PR took 0.12 s vs 0.08 s on main (49% slower).
📖 Linting Realm with this PR took 0.19 s vs 0.21 s on main (9% faster).
📖 Linting Sourcery with this PR took 0.19 s vs 0.17 s on main (11% slower).
📖 Linting Swift with this PR took 0.21 s vs 0.21 s on main (0% slower).
📖 Linting SwiftLintPerformanceTests with this PR took 3.73 s vs 3.73 s on main (0% slower).
📖 Linting VLC with this PR took 0.19 s vs 0.18 s on main (5% slower).
📖 Linting Wire with this PR took 0.79 s vs 0.79 s on main (0% slower).
📖 Linting WordPress with this PR took 0.61 s vs 0.6 s on main (1% slower).

Generated by 🚫 Danger

@CraigSiemens
CraigSiemens force-pushed the fix-legacy-swiftui-aspect-ratio-lost-whitespace branch from 158882e to 41a7b0a Compare August 24, 2026 15:35

@SimplyDanny SimplyDanny left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @CraigSiemens!

Comment thread CHANGELOG.md Outdated
@SimplyDanny
SimplyDanny enabled auto-merge (squash) August 24, 2026 17:37
@SimplyDanny
SimplyDanny merged commit 8cd0222 into realm:main Aug 24, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Applied fix for legacy_swiftui_aspect_ratio violations doesn't preserve newlines

3 participants