Skip to content

Commit

Permalink
swiftlint build plugin (#1318)
Browse files Browse the repository at this point in the history
  • Loading branch information
mallexxx authored Dec 18, 2023
1 parent 2982a23 commit 751f90c
Show file tree
Hide file tree
Showing 105 changed files with 443 additions and 329 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
steps:
- uses: actions/checkout@v3
- name: SwiftLint
uses: docker://norionomura/swiftlint:0.53.0
uses: docker://norionomura/swiftlint:0.54.0
with:
args: swiftlint --reporter github-actions-logging --strict

Expand Down Expand Up @@ -133,6 +133,7 @@ jobs:
-scheme "${{ matrix.scheme }}" \
-derivedDataPath "DerivedData" \
-configuration "CI" \
-skipPackagePluginValidation \
ENABLE_TESTABILITY=true \
ONLY_ACTIVE_ARCH=${{ matrix.active-arch }} \
"-skip-testing:${{ matrix.integration-tests-target }}" \
Expand All @@ -146,6 +147,7 @@ jobs:
-scheme "${{ matrix.scheme }}" \
-derivedDataPath "DerivedData" \
-configuration "CI" \
-skipPackagePluginValidation \
ENABLE_TESTABILITY=true \
ONLY_ACTIVE_ARCH=${{ matrix.active-arch }} \
"-only-testing:${{ matrix.integration-tests-target }}" \
Expand Down Expand Up @@ -320,6 +322,7 @@ jobs:
-scheme "${{ matrix.scheme }}" \
-derivedDataPath "DerivedData" \
-configuration "Release" \
-skipPackagePluginValidation \
| tee release-xcodebuild.log \
| xcbeautify
Expand Down
53 changes: 0 additions & 53 deletions .swiftlint.tests.yml

This file was deleted.

12 changes: 6 additions & 6 deletions .swiftlint.yml
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
allow_zero_lintable_files: true

disabled_rules:
- no_space_in_method_call
- multiple_closures_with_trailing_closure
Expand All @@ -9,9 +11,12 @@ disabled_rules:
- implicit_getter
- function_parameter_count
- trailing_comma
- nesting
- opening_brace

opt_in_rules:
- file_header
- explicit_init

custom_rules:
explicit_non_final_class:
Expand Down Expand Up @@ -41,15 +46,13 @@ file_length:
type_body_length:
warning: 500
error: 500
nesting:
type_level: 2
large_tuple:
warning: 4
error: 5
file_header:
required_pattern: |
\/\/
\/\/ .*?\.swift
\/\/ SWIFTLINT_CURRENT_FILENAME
\/\/
\/\/ Copyright © \d{4} DuckDuckGo\. All rights reserved\.
\/\/
Expand All @@ -69,11 +72,8 @@ file_header:
# General Config
excluded:
- DuckDuckGo/Common/Localizables/UserText.swift
- UnitTests
- IntegrationTests
- LocalPackages/*/Package.swift
- release
- UITests
- vendor
- DerivedData
- .ruby-lsp
Loading

0 comments on commit 751f90c

Please sign in to comment.