Skip to content

Use start position of closure to check if rule is disabled#6458

Merged
SimplyDanny merged 1 commit intomainfrom
trailing-closure
Jan 23, 2026
Merged

Use start position of closure to check if rule is disabled#6458
SimplyDanny merged 1 commit intomainfrom
trailing-closure

Conversation

@SimplyDanny
Copy link
Collaborator

@SimplyDanny SimplyDanny commented Jan 23, 2026 •
edited
Loading

Resolves #6451.

@SwiftLintBot
Copy link

SwiftLintBot commented Jan 23, 2026 •
edited
Loading

19 Messages
📖 Building this branch resulted in a binary size of 27231.88 KiB vs 27231.32 KiB when built on main (0% larger).
📖 Linting Aerial with this PR took 0.82 s vs 0.8 s on main (2% slower).
📖 Linting Alamofire with this PR took 1.05 s vs 1.07 s on main (1% faster).
📖 Linting Brave with this PR took 7.28 s vs 7.3 s on main (0% faster).
📖 Linting DuckDuckGo with this PR took 24.69 s vs 24.73 s on main (0% faster).
📖 Linting Firefox with this PR took 11.72 s vs 11.85 s on main (1% faster).
📖 Linting Kickstarter with this PR took 8.06 s vs 8.11 s on main (0% faster).
📖 Linting Moya with this PR took 0.45 s vs 0.47 s on main (4% faster).
📖 Linting NetNewsWire with this PR took 2.48 s vs 2.49 s on main (0% faster).
📖 Linting Nimble with this PR took 0.69 s vs 0.67 s on main (2% slower).
📖 Linting PocketCasts with this PR took 7.41 s vs 7.46 s on main (0% faster).
📖 Linting Quick with this PR took 0.45 s vs 0.45 s on main (0% slower).
📖 Linting Realm with this PR took 3.33 s vs 3.28 s on main (1% slower).
📖 Linting Sourcery with this PR took 1.87 s vs 1.88 s on main (0% faster).
📖 Linting Swift with this PR took 4.51 s vs 4.54 s on main (0% faster).
📖 Linting SwiftLintPerformanceTests with this PR took 0.36 s vs 0.35 s on main (2% slower).
📖 Linting VLC with this PR took 1.15 s vs 1.19 s on main (3% faster).
📖 Linting Wire with this PR took 18.68 s vs 18.78 s on main (0% faster).
📖 Linting WordPress with this PR took 12.44 s vs 12.45 s on main (0% faster).

Generated by 🚫 Danger

@SimplyDanny SimplyDanny merged commit 987f360 into main Jan 23, 2026
27 checks passed
@SimplyDanny SimplyDanny deleted the trailing-closure branch January 23, 2026 19:52
nadeemnali pushed a commit to nadeemnali/SwiftLint that referenced this pull request Feb 26, 2026
@SimplyDanny @nadeemnali
Use start position of closure to check if rule is disabled (realm#6458)
7fbbcd5
nadeemnali pushed a commit to nadeemnali/SwiftLint that referenced this pull request Feb 26, 2026
@SimplyDanny @nadeemnali
Use start position of closure to check if rule is disabled (realm#6458)
d9667cd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

trailing_closure autocorrect ignores swiftlint:disable:this

2 participants

@SimplyDanny @SwiftLintBot