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

swiftlint build plugin #1318

Merged
merged 26 commits into from
Dec 18, 2023
Merged

swiftlint build plugin #1318

merged 26 commits into from
Dec 18, 2023

Conversation

mallexxx
Copy link
Collaborator

@mallexxx mallexxx commented Jun 30, 2023

Task/Issue URL: https://app.asana.com/0/1205944207430351/1204377796839424/f
BSK PR: duckduckgo/BrowserServicesKit#393
iOS PR: duckduckgo/iOS#2227
CC: @samsymons

Description:

  • Implemented

Steps to test this PR:

  1. Validate linter works without issues both locally and on CI

Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@github-actions
Copy link
Contributor

github-actions bot commented Jun 30, 2023

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS against fabd2ac

@mallexxx mallexxx changed the title [WIP] swiftlint build plugin swiftlint build plugin Dec 6, 2023
@mallexxx mallexxx requested a review from samsymons December 6, 2023 13:47
@mallexxx mallexxx force-pushed the alex/swiftlint-plugin branch from fbfe587 to 6defa8d Compare December 6, 2023 14:21
@ayoy ayoy changed the base branch from develop to main December 6, 2023 16:06
Comment on lines -52 to +55
\/\/ .*?\.swift
\/\/ SWIFTLINT_CURRENT_FILENAME
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice

Copy link
Collaborator

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

From testing locally this looks good. I believe this will undo a recent change where SwiftLint will also run on non-debug builds, but that's a worthwhile tradeoff if it means a consistent SwiftLint experience and a pinned version.

cc @ayoy to make sure this sounds good to him too.

@mallexxx
Copy link
Collaborator Author

Thanks Sam, I believe it's not a problem since the Xcode release build flow is broken anyways and linter won't run for console or CI builds, and won't raise fatal issues for Xcode builds either

@mallexxx mallexxx merged commit 751f90c into main Dec 18, 2023
14 checks passed
@mallexxx mallexxx deleted the alex/swiftlint-plugin branch December 18, 2023 08:09
samsymons added a commit that referenced this pull request Dec 19, 2023
* main:
  DBP: Fix unreliable date tests (#1981)
  Add search retention pixel for NetP (#1964)
  Sabrina/sync e2e tests (#1959)
  swiftlint build plugin (#1318)
  VPN Geoswitching - initial draft (#1978)
  Add additional VPN startup pixels (#1975)
  Updates to Autofill Logins copy (#1924)
  Bump version to 1.69.0 (95)
  Remove the reconnect/disconnect logic from the connection tester (#1970)
  Set marketing version to 1.69.0
  Update embedded files
  DBP: Send internal user param for dbp waitlist pixels (#1972)
  Move release task to proper section in Code Freeze workflow (#1977)
  drop Main.storyboard (#1944)
  Add GHA workflow to cut release branch (#1976)
  Move DBP tests into main target (#1974)
  Use static date for PixelKit tests (#1973)
samsymons added a commit that referenced this pull request Dec 21, 2023
# By Dominik Kapusta (41) and others
# Via Dominik Kapusta (9) and others
* main: (138 commits)
  Make sure when we set custom config url, we don't expect etag in return (#1994)
  Add PixelKit source parameter (#1989)
  Fix internal user toggling (#2000)
  Show alert and display warning icon in Sync Settings when data syncing is disabled (#1996)
  DBP: Integrate subscription account authentication to DBP (#1995)
  Improve bookmarks html reader (#1986)
  Add Sync feature flags (#1992)
  Add daily stats pixel (#1993)
  Do not reload DBP tab when switching to it (#1942)
  Fix: external application requests via redirect URLs shows wrong origin. (#1900)
  Update clean-app.sh to work on macOS Sonoma and include NetP containers (#1988)
  Fix: "SwiftLintPlugin" must be enabled before it can be used (#1987)
  Prevent VPN server list persistence failures (#1985)
  add test can remove data (#1980)
  Remove VPN upgrade card (#1983)
  Fix low-res VPN warning asset (#1984)
  DBP: Fix unreliable date tests (#1981)
  Add search retention pixel for NetP (#1964)
  Sabrina/sync e2e tests (#1959)
  swiftlint build plugin (#1318)
  ...

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	DuckDuckGo/Application/AppDelegate.swift
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.

2 participants