Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Commit a8ef60f

Browse files
crazytonylicrazytonyli
committed
Merge branch 'trunk' into tonyli-remove-alamofire
2 parents 5f818b3 + 12ca8d5 commit a8ef60f

20 files changed

+127
-29
lines changed

‎.buildkite/pipeline.yml‎

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Nodes with values to reuse in the pipeline.
22
common_params:
33
plugins: &common_plugins
4-
- automattic/a8c-ci-toolkit#3.0.1
4+
- automattic/a8c-ci-toolkit#3.1.0
55
# Common environment values to use with the `env` key.
66
env: &common_env
77
IMAGE_ID: xcode-15.0.1
@@ -33,8 +33,17 @@ steps:
3333
plugins: *common_plugins
3434

3535
#################
36-
# Lint
36+
# Linters
3737
#################
38+
- label: ":swift: SwiftLint"
39+
command: run_swiftlint --strict
40+
plugins: *common_plugins
41+
notify:
42+
- github_commit_status:
43+
context: "SwiftLint"
44+
agents:
45+
queue: "default"
46+
3847
- label: "🧹 Lint"
3948
key: "lint"
4049
command: |

‎.gitignore‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,3 +96,6 @@ vendor/
9696
# https://github.com/johnno1962/injectionforxcode
9797

9898
iOSInjectionProject/
99+
100+
# SwiftLint Remote Config Cache
101+
.swiftlint/RemoteConfigCache

‎.rubocop.yml‎

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,15 @@
22
AllCops:
33
NewCops: enable
44

5+
# Override the maximum line length
6+
Layout/LineLength:
7+
Max: 160
8+
59
# Allow the Podspec filename to match the project
610
Naming/FileName:
711
Exclude:
812
- 'WordPressKit.podspec'
913

10-
# Override the maximum line length
11-
Layout/LineLength:
12-
Max: 160
14+
Metrics/BlockLength:
15+
Exclude:
16+
- 'fastlane/Fastfile'

‎.swiftlint.yml‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
swiftlint_version: 0.54.0
2+
3+
parent_config: https://raw.githubusercontent.com/Automattic/swiftlint-config/0f8ab6388bd8d15a04391825ab125f80cfb90704/.swiftlint.yml
4+
remote_timeout: 10.0
5+
6+
opt_in_rules:
7+
- overridden_super_call
8+
- discarded_notification_center_observer
9+
- weak_delegate
10+
- vertical_whitespace
11+
- duplicate_imports
12+
13+
overridden_super_call:
14+
severity: error
15+
16+
discarded_notification_center_observer:
17+
severity: error
18+
19+
weak_delegate:
20+
severity: error

‎Podfile‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,12 @@ APP_IOS_DEPLOYMENT_TARGET = Gem::Version.new('13.0')
99

1010
platform :ios, APP_IOS_DEPLOYMENT_TARGET
1111

12+
def swiftlint_version
13+
require 'yaml'
14+
15+
YAML.load_file('.swiftlint.yml')['swiftlint_version']
16+
end
17+
1218
def wordpresskit_pods
1319
pod 'WordPressShared', '~> 2.0.0-beta.2'
1420
pod 'NSObject-SafeExpectations', '~> 0.0.4'
@@ -34,6 +40,10 @@ target 'WordPressKitTests' do
3440
pod 'Alamofire', '~> 5.0'
3541
end
3642

43+
abstract_target 'Tools' do
44+
pod 'SwiftLint', swiftlint_version
45+
end
46+
3747
# Let Pods targets inherit deployment target from the app
3848
post_install do |installer|
3949
installer.pods_project.targets.each do |target|
@@ -43,4 +53,9 @@ post_install do |installer|
4353
configuration.build_settings.delete(ios_deployment_key) if pod_ios_deployment_target <= APP_IOS_DEPLOYMENT_TARGET
4454
end
4555
end
56+
57+
yellow_marker = "\033[33m"
58+
reset_marker = "\033[0m"
59+
puts "#{yellow_marker}The abstract target warning below is expected. Feel free to ignore
60+
it.#{reset_marker}"
4661
end

‎Podfile.lock‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ PODS:
1717
- OHHTTPStubs/OHPathHelpers (9.1.0)
1818
- OHHTTPStubs/Swift (9.1.0):
1919
- OHHTTPStubs/Default
20+
- SwiftLint (0.54.0)
2021
- UIDeviceIdentifier (2.3.0)
2122
- WordPressShared (2.0.1)
2223
- wpxmlrpc (0.10.0)
@@ -27,6 +28,7 @@ DEPENDENCIES:
2728
- OCMock (~> 3.4)
2829
- OHHTTPStubs (~> 9.0)
2930
- OHHTTPStubs/Swift (~> 9.0)
31+
- SwiftLint (= 0.54.0)
3032
- UIDeviceIdentifier (~> 2.0)
3133
- WordPressShared (~> 2.0.0-beta.2)
3234
- wpxmlrpc (~> 0.10.0)
@@ -37,6 +39,7 @@ SPEC REPOS:
3739
- NSObject-SafeExpectations
3840
- OCMock
3941
- OHHTTPStubs
42+
- SwiftLint
4043
- UIDeviceIdentifier
4144
- WordPressShared
4245
- wpxmlrpc
@@ -46,10 +49,11 @@ SPEC CHECKSUMS:
4649
NSObject-SafeExpectations: c01c8881cbd97efad6f668286b913cd0b7d62ab5
4750
OCMock: 300b1b1b9155cb6378660b981c2557448830bdc6
4851
OHHTTPStubs: 90eac6d8f2c18317baeca36698523dc67c513831
52+
SwiftLint: c1de071d9d08c8aba837545f6254315bc900e211
4953
UIDeviceIdentifier: 442b65b4ff1832d4ca9c2a157815cb29ad981b17
5054
WordPressShared: f93f99269258b46dad04f4e4dbf540ce2e5c1e66
5155
wpxmlrpc: 68db063041e85d186db21f674adf08d9c70627fd
5256

53-
PODFILE CHECKSUM: 30cce0948b823763e81a644e832798bb48d02cd1
57+
PODFILE CHECKSUM: 64af6d71574c7a92d01a9446aa874e066917ebe5
5458

5559
COCOAPODS: 1.14.3

‎WordPressKit.xcodeproj/project.pbxproj‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2829,6 +2829,7 @@
28292829
9368C7761EC5EF1B0092CE8E /* Sources */,
28302830
9368C7771EC5EF1B0092CE8E /* Frameworks */,
28312831
9368C7791EC5EF1B0092CE8E /* Resources */,
2832+
3F391E242B577AD7007975C4 /* SwiftLint */,
28322833
);
28332834
buildRules = (
28342835
);
@@ -3247,6 +3248,25 @@
32473248
shellScript = "diff \"${PODS_PODFILE_DIR_PATH}/Podfile.lock\" \"${PODS_ROOT}/Manifest.lock\" > /dev/null\nif [ $? != 0 ] ; then\n # print error to STDERR\n echo \"error: The sandbox is not in sync with the Podfile.lock. Run 'pod install' or update your CocoaPods installation.\" >&2\n exit 1\nfi\n# This output is used by Xcode 'outputs' to avoid re-running this script phase.\necho \"SUCCESS\" > \"${SCRIPT_OUTPUT_FILE_0}\"\n";
32483249
showEnvVarsInLog = 0;
32493250
};
3251+
3F391E242B577AD7007975C4 /* SwiftLint */ = {
3252+
isa = PBXShellScriptBuildPhase;
3253+
alwaysOutOfDate = 1;
3254+
buildActionMask = 2147483647;
3255+
files = (
3256+
);
3257+
inputFileListPaths = (
3258+
);
3259+
inputPaths = (
3260+
);
3261+
name = SwiftLint;
3262+
outputFileListPaths = (
3263+
);
3264+
outputPaths = (
3265+
);
3266+
runOnlyForDeploymentPostprocessing = 0;
3267+
shellPath = /bin/sh;
3268+
shellScript = "./Pods/SwiftLint/swiftlint lint\n";
3269+
};
32503270
50C31E8D1B97D5FA0D543935 /* [CP] Check Pods Manifest.lock */ = {
32513271
isa = PBXShellScriptBuildPhase;
32523272
buildActionMask = 2147483647;

‎WordPressKit/AccountServiceRemoteREST+SocialService.swift‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ extension AccountServiceRemoteREST {
2222
connectParameters: [String: AnyObject]? = nil,
2323
oAuthClientID: String,
2424
oAuthClientSecret: String,
25-
success:@escaping (() -> Void),
26-
failure:@escaping ((NSError) -> Void)) {
25+
success: @escaping (() -> Void),
26+
failure: @escaping ((NSError) -> Void)) {
2727
let path = self.path(forEndpoint: "me/social-login/connect", withVersion: ._1_1)
2828

2929
var params = [
@@ -65,7 +65,7 @@ extension AccountServiceRemoteREST {
6565
/// - oAuthClientSecret The WPCOM REST API client secret.
6666
/// - success The block that will be executed on success.
6767
/// - failure The block that will be executed on failure.
68-
public func disconnectFromSocialService(_ service: SocialServiceName, oAuthClientID: String, oAuthClientSecret: String, success:@escaping(() -> Void), failure:@escaping((NSError) -> Void)) {
68+
public func disconnectFromSocialService(_ service: SocialServiceName, oAuthClientID: String, oAuthClientSecret: String, success: @escaping(() -> Void), failure: @escaping((NSError) -> Void)) {
6969
let path = self.path(forEndpoint: "me/social-login/disconnect", withVersion: ._1_1)
7070
let params = [
7171
"client_id": oAuthClientID,

‎WordPressKit/ChecksumUtil.swift‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import Foundation
22

33
public class ChecksumUtil {
44

5-
/// Generates a checksum based on the encoded keys.
5+
/// Generates a checksum based on the encoded keys.
66
static func checksum<T>(from codable: T) -> String where T: Encodable {
77
let encoder = JSONEncoder()
88
encoder.outputFormatting = .sortedKeys

‎WordPressKit/GravatarServiceRemote.swift‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ open class GravatarServiceRemote {
1515
/// - success: A success block.
1616
/// - failure: A failure block.
1717
///
18-
open func fetchProfile(_ email: String, success:@escaping ((_ profile: RemoteGravatarProfile) -> Void), failure:@escaping ((_ error: Error?) -> Void)) {
18+
open func fetchProfile(_ email: String, success: @escaping ((_ profile: RemoteGravatarProfile) -> Void), failure: @escaping ((_ error: Error?) -> Void)) {
1919
guard let hash = (email as NSString).md5() else {
2020
assertionFailure()
2121
return
@@ -31,7 +31,7 @@ open class GravatarServiceRemote {
3131
/// - success: A success block.
3232
/// - failure: A failure block.
3333
///
34-
open func fetchProfile(hash: String, success:@escaping ((_ profile: RemoteGravatarProfile) -> Void), failure:@escaping ((_ error: Error?) -> Void)) {
34+
open func fetchProfile(hash: String, success: @escaping ((_ profile: RemoteGravatarProfile) -> Void), failure: @escaping ((_ error: Error?) -> Void)) {
3535
let path = baseGravatarURL + hash + ".json"
3636
guard let targetURL = URL(string: path) else {
3737
assertionFailure()

0 commit comments

Comments
 (0)