Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds opt-in discouraged_optional_boolean rule #2013

Merged
merged 4 commits into from
Feb 3, 2018

Conversation

ornithocoder
Copy link
Contributor

@ornithocoder ornithocoder commented Jan 21, 2018

Implements #2011.

@SwiftLintBot
Copy link

SwiftLintBot commented Jan 22, 2018

305 Warnings
⚠️ Big PR
⚠️ This PR introduced a violation in Alamofire: /Users/distiller/project/osscheck/Alamofire/Tests/UploadTests.swift:347:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Alamofire: /Users/distiller/project/osscheck/Alamofire/Tests/UploadTests.swift:392:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Alamofire: /Users/distiller/project/osscheck/Alamofire/Tests/UploadTests.swift:444:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Alamofire: /Users/distiller/project/osscheck/Alamofire/Tests/UploadTests.swift:491:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Alamofire: /Users/distiller/project/osscheck/Alamofire/Tests/UploadTests.swift:556:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Client/Frontend/AuthenticationManager/AuthenticationSettingsViewController.swift:107:93: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Client/Frontend/AuthenticationManager/AuthenticationSettingsViewController.swift:172:18: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Client/Application/AppDelegate.swift:828:20: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Client/Frontend/ContentBlocker/ContentBlockerHelper.swift:29:60: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Client/Frontend/Settings/AdvanceAccountSettingViewController.swift:163:44: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Client/Frontend/Settings/SettingsTableViewController.swift:98:156: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Providers/NSUserDefaultsPrefs.swift:76:52: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Shared/Extensions/NSURLExtensions.swift:84:57: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Shared/Prefs.swift:53:47: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Shared/Prefs.swift:124:52: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Storage/Site.swift:46:43: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Storage/Site.swift:52:57: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Firefox: /Users/distiller/project/osscheck/Firefox/Storage/SQL/SQLiteHistoryFactories.swift:20:27: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:8:35: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:10:41: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:11:36: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:12:43: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:13:37: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:14:40: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:18:40: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:21:35: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:23:39: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:24:36: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:97:63: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:99:66: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:100:61: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:103:63: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:104:66: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:109:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:113:63: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:115:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:116:64: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:121:57: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/DiscoveryParams.swift:139:60: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:5:54: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:21:60: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:29:55: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:37:62: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:45:56: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:53:59: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:85:59: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:109:54: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:125:58: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/DiscoveryParamsLenses.swift:133:55: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/Project.PersonalizationLenses.swift:17:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/Project.PersonalizationLenses.swift:23:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/ProjectLenses.swift:293:39: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/ProjectLenses.swift:297:39: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/PushEnvelopeLenses.swift:17:55: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NewsletterSubscriptionsLenses.swift:5:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NewsletterSubscriptionsLenses.swift:11:66: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NewsletterSubscriptionsLenses.swift:17:70: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NewsletterSubscriptionsLenses.swift:23:67: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NewsletterSubscriptionsLenses.swift:29:66: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NewsletterSubscriptionsLenses.swift:35:67: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:5:59: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:15:59: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:26:59: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:37:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:46:60: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:56:62: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:66:58: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:76:64: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:86:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:95:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:104:65: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:113:71: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:122:66: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/User.NotificationsLenses.swift:132:64: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:12:54: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:26:45: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:68:43: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:99:31: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:103:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:107:36: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:111:33: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:115:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:119:33: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:125:35: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:129:35: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:133:35: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:137:41: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:141:36: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:145:38: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:149:40: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:153:34: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:157:41: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:161:41: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:165:41: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:169:47: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:173:42: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/lenses/UserLenses.swift:177:40: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/Project.swift:120:27: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/Project.swift:121:27: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/PushEnvelope.swift:8:26: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/Update.swift:9:24: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/Update.swift:19:23: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:7:33: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:9:24: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:15:22: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:25:22: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:26:23: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:27:27: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:28:24: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:29:23: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:30:24: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:34:26: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:35:26: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:36:26: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:37:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:38:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:39:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:40:32: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:41:38: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ This PR introduced a violation in Kickstarter: /Users/distiller/project/osscheck/Kickstarter/KsApi/models/User.swift:42:33: warning: Discouraged Optional Boolean Violation: Prefer non-optional booleans over optional booleans. (discouraged_optional_boolean)
⚠️ Danger found 305 violations with this PR. Due to GitHub's max issue comment size, the number shown has been truncated to 121.
12 Messages
📖 Linting Aerial with this PR took 0.52s vs 0.41s on master (26% slower)
📖 Linting Alamofire with this PR took 3.91s vs 3.48s on master (12% slower)
📖 Linting Firefox with this PR took 14.73s vs 13.57s on master (8% slower)
📖 Linting Kickstarter with this PR took 22.72s vs 22.45s on master (1% slower)
📖 Linting Moya with this PR took 1.84s vs 2.01s on master (8% faster)
📖 Linting Nimble with this PR took 1.97s vs 2.1s on master (6% faster)
📖 Linting Quick with this PR took 0.57s vs 0.62s on master (8% faster)
📖 Linting Realm with this PR took 3.6s vs 3.67s on master (1% faster)
📖 Linting SourceKitten with this PR took 1.15s vs 1.16s on master (0% faster)
📖 Linting Sourcery with this PR took 5.14s vs 5.17s on master (0% faster)
📖 Linting Swift with this PR took 14.39s vs 14.55s on master (1% faster)
📖 Linting WordPress with this PR took 14.92s vs 14.64s on master (1% slower)

Generated by 🚫 Danger

@codecov-io
Copy link

codecov-io commented Jan 22, 2018

Codecov Report

Merging #2013 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2013      +/-   ##
==========================================
+ Coverage   89.65%   89.67%   +0.01%     
==========================================
  Files         254      256       +2     
  Lines       14843    14861      +18     
  Branches      965      968       +3     
==========================================
+ Hits        13308    13327      +19     
+ Misses       1516     1515       -1     
  Partials       19       19
Impacted Files Coverage Δ
...Rules/DiscouragedOptionalBooleanRuleExamples.swift 100% <100%> (ø)
...amework/Rules/DiscouragedOptionalBooleanRule.swift 100% <100%> (ø)
Tests/SwiftLintFrameworkTests/RulesTests.swift 100% <100%> (ø) ⬆️
...SwiftLintFramework/Extensions/File+SwiftLint.swift 95.11% <0%> (+0.35%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 953b1dd...404c5e1. Read the comment docs.

@ornithocoder ornithocoder changed the title [WIP] Adds opt-in discouraged_optional_boolean rule Adds opt-in discouraged_optional_boolean rule Jan 22, 2018
@@ -74,6 +74,10 @@ class RulesTests: XCTestCase {
verifyRule(DiscouragedObjectLiteralRule.description)
}

func testDiscouragedOptionalBoolean() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

You need to run Sourcery to update the tests list to be run on Linux

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done :-) Sorry about that, @marcelofabri. Thanks.

Copy link
Collaborator

@marcelofabri marcelofabri left a comment

Choose a reason for hiding this comment

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

Thanks! 💯

@marcelofabri marcelofabri merged commit d4346d2 into realm:master Feb 3, 2018
@ornithocoder ornithocoder deleted the optional_boolean branch February 4, 2018 08:23
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.

4 participants