search

realm / SwiftLint

A tool to enforce Swift style and conventions.
https://realm.github.io/SwiftLint
MIT License
18.46k stars 2.2k forks source link

all: fix some comments #5504

Closed deterclosed closed 3 months ago

SwiftLintBot commented 3 months ago
16 Warnings
:warning: This PR introduced a violation in Brave: /Sources/Shared/URIFixup.swift:89:41: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in Brave: /Sources/Shared/URIFixup.swift:89:8: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in Brave: /Sources/Shared/URIFixup.swift:93:8: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:124:61: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:124:7: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:125:62: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:125:8: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:129:62: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:129:7: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:130:63: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:130:8: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in NetNewsWire: /buildscripts/VerifyNoBS.swift:94:27: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in PocketCasts: /podcasts/SwiftUI/HighlightedText.swift:133:16: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in PocketCasts: /podcasts/SwiftUI/HighlightedText.swift:141:20: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in WordPress: /WordPress/WordPressTest/NotificationSettingsServiceTests.swift:31:20: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:warning: This PR introduced a violation in WordPress: /WordPress/WordPressTest/OHHTTPStubs+Helpers.swift:9:20: warning: Contains over Range Comparison to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
33 Messages
:book: Linting Aerial with this PR took 0.8s vs 0.83s on main (3% faster)
:book: Linting Alamofire with this PR took 1.19s vs 1.12s on main (6% slower)
:book: Linting Brave with this PR took 6.51s vs 6.56s on main (0% faster)
:book: Linting DuckDuckGo with this PR took 3.54s vs 3.55s on main (0% faster)
:book: Linting Firefox with this PR took 8.87s vs 8.92s on main (0% faster)
:book: Linting Kickstarter with this PR took 8.09s vs 8.0s on main (1% slower)
:book: Linting Moya with this PR took 0.5s vs 0.48s on main (4% slower)
:book: Linting NetNewsWire with this PR took 2.4s vs 2.38s on main (0% slower)
:book: Linting Nimble with this PR took 0.69s vs 0.69s on main (0% slower)
:book: Linting PocketCasts with this PR took 6.79s vs 6.81s on main (0% faster)
:book: Linting Quick with this PR took 0.33s vs 0.33s on main (0% slower)
:book: Linting Realm with this PR took 4.02s vs 4.0s on main (0% slower)
:book: Linting Sourcery with this PR took 2.07s vs 2.03s on main (1% slower)
:book: Linting Swift with this PR took 3.84s vs 3.83s on main (0% slower)
:book: Linting VLC with this PR took 1.14s vs 1.1s on main (3% slower)
:book: Linting Wire with this PR took 14.14s vs 14.08s on main (0% slower)
:book: Linting WordPress with this PR took 9.41s vs 9.49s on main (0% faster)
:book: This PR fixed a violation in Brave: /Sources/Shared/URIFixup.swift:89:41: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in Brave: /Sources/Shared/URIFixup.swift:89:8: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in Brave: /Sources/Shared/URIFixup.swift:93:8: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:124:61: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:124:7: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:125:62: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:125:8: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:129:62: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:129:7: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:130:63: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /FoundationExtras/Sources/FoundationExtras/Data+RSCore.swift:130:8: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in NetNewsWire: /buildscripts/VerifyNoBS.swift:94:27: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in PocketCasts: /podcasts/SwiftUI/HighlightedText.swift:133:16: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in PocketCasts: /podcasts/SwiftUI/HighlightedText.swift:141:20: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in WordPress: /WordPress/WordPressTest/NotificationSettingsServiceTests.swift:31:20: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)
:book: This PR fixed a violation in WordPress: /WordPress/WordPressTest/OHHTTPStubs+Helpers.swift:9:20: warning: Contains over Range Comparision to Nil Violation: Prefer contains over range(of:) != nil and range(of:) == nil (contains_over_range_nil_comparison)

Generated by :no_entry_sign: Danger

SimplyDanny commented 3 months ago

Thank you!