Skip to content

Commit

Permalink
Fix linting issues after CI SwiftLint update (#2055)
Browse files Browse the repository at this point in the history
Co-authored-by: Chris Brind <[email protected]>
  • Loading branch information
graeme and brindy authored Sep 29, 2023
1 parent 76db01b commit 9028834
Show file tree
Hide file tree
Showing 9 changed files with 2 additions and 41 deletions.
4 changes: 0 additions & 4 deletions DuckDuckGo/AutofillListItemTableViewCell.swift
Original file line number Diff line number Diff line change
Expand Up @@ -118,10 +118,6 @@ class AutofillListItemTableViewCell: UITableViewCell {
contentStackView.bottomAnchor.constraint(equalTo: margins.bottomAnchor)
])
}

override func setSelected(_ selected: Bool, animated: Bool) {
super.setSelected(selected, animated: animated)
}

private func setupContentView(with item: AutofillLoginListItemViewModel) {
titleLabel.text = item.title
Expand Down
2 changes: 1 addition & 1 deletion DuckDuckGo/DaxDialogs.swift
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ final class DaxDialogs {
case .siteIsMajorTracker, .siteOwnedByMajorTracker:
settings.browsingMajorTrackingSiteShown = flag
settings.browsingWithoutTrackersShown = flag
}
}
}

struct BrowsingSpec: Equatable {
Expand Down
4 changes: 0 additions & 4 deletions DuckDuckGo/KeychainItemsDebugViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -115,10 +115,6 @@ private enum SecClass: CaseIterable {

class KeychainItemsDebugViewController: UITableViewController {

override func viewDidLoad() {
super.viewDidLoad()
}

override func numberOfSections(in tableView: UITableView) -> Int {
return SecClass.allCases.count
}
Expand Down
3 changes: 0 additions & 3 deletions DuckDuckGo/OnboardingDefaultBroswerViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,4 @@ class OnboardingDefaultBroswerViewController: OnboardingContentViewController {
super.onContinuePressed(navigationHandler: navigationHandler)
}

override func onSkipPressed(navigationHandler: @escaping () -> Void) {
super.onSkipPressed(navigationHandler: navigationHandler)
}
}
4 changes: 0 additions & 4 deletions DuckDuckGo/SaveLoginViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -61,10 +61,6 @@ class SaveLoginViewController: UIViewController {
viewModel?.viewControllerDidAppear()
}

override func viewDidLayoutSubviews() {
super.viewDidLayoutSubviews()
}

override func viewDidDisappear(_ animated: Bool) {
super.viewDidDisappear(animated)

Expand Down
4 changes: 0 additions & 4 deletions DuckDuckGo/SettingsViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -544,10 +544,6 @@ class SettingsViewController: UITableViewController {
}
}

override func tableView(_ tableView: UITableView, titleForFooterInSection section: Int) -> String? {
return super.tableView(tableView, titleForFooterInSection: section)
}

override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
let rows = super.tableView(tableView, numberOfRowsInSection: section)
if section == appearanceSectionIndex && textSizeCell.isHidden {
Expand Down
8 changes: 0 additions & 8 deletions DuckDuckGoTests/PrivacyIconLogicTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -33,14 +33,6 @@ class PrivacyIconLogicTests: XCTestCase {
static let ddgMainURL = URL(string: "https://duckduckgo.com")!
static let ddgSupportURL = URL(string: "https://duckduckgo.com/email/settings/support")!

override func setUp() {
super.setUp()
}

override func tearDown() {
super.tearDown()
}

func testPrivacyIconIsShieldForPageURL() {
let url = PrivacyIconLogicTests.insecurePageURL
let icon = PrivacyIconLogic.privacyIcon(for: url)
Expand Down
8 changes: 0 additions & 8 deletions DuckDuckGoTests/TrackerAnimationLogicTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -30,14 +30,6 @@ class TrackerAnimationLogicTests: XCTestCase {

static let pageURL = URL(string: "https://example.com")!

override func setUp() {
super.setUp()
}

override func tearDown() {
super.tearDown()
}

func testAnimationLogicToAnimateTrackersIfAnyBlocked() {
let trackerInfo = makeBlockedTrackerInfo(pageURL: Self.pageURL)
XCTAssertTrue(TrackerAnimationLogic.shouldAnimateTrackers(for: trackerInfo))
Expand Down
6 changes: 1 addition & 5 deletions IntegrationTests/AtbServerTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,7 @@ class AtbServerTests: XCTestCase {
loader = StatisticsLoader(statisticsStore: store)

}

override func tearDown() {
super.tearDown()
}


func testExtiCall() {

let waitForCompletion = expectation(description: "wait for completion")
Expand Down

0 comments on commit 9028834

Please sign in to comment.