Skip to content

Commit 69e5445

Browse files
authored
Merge pull request #524 from 417-72KI/drop-swift-5.3
2 parents 9bf6736 + 2810779 commit 69e5445

File tree

5 files changed

+21
-36
lines changed

5 files changed

+21
-36
lines changed

.github/workflows/ci.yml

+14-5
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,18 @@ on:
77
- main
88
- master
99

10+
concurrency:
11+
group: ${{ github.ref }}-${{ github.workflow }}
12+
cancel-in-progress: true
13+
1014
jobs:
1115
test-on-macos-11:
1216
name: Test on macOS 11
1317
runs-on: macOS-11
1418
strategy:
19+
fail-fast: false
1520
matrix:
16-
xcode: ["12.4", "12.5.1", "13.0"]
21+
xcode: ["12.5.1", "13.2.1"]
1722
steps:
1823
- uses: actions/checkout@v2
1924

@@ -50,8 +55,9 @@ jobs:
5055
name: Test dependencies resolver
5156
runs-on: macOS-11
5257
strategy:
58+
fail-fast: false
5359
matrix:
54-
xcode: ["12.4", "12.5.1", "13.0"]
60+
xcode: ["12.5.1", "13.2.1"]
5561
steps:
5662
- uses: actions/checkout@v2
5763

@@ -90,8 +96,9 @@ jobs:
9096
name: Test on Linux
9197
runs-on: ubuntu-latest
9298
strategy:
99+
fail-fast: false
93100
matrix:
94-
swift: ["5.3.3", "5.5.2"]
101+
swift: ["5.4.3", "5.5.3"]
95102
steps:
96103
- uses: actions/checkout@v2
97104
- uses: actions/setup-node@v2
@@ -128,8 +135,9 @@ jobs:
128135
name: Test without SPM on macOS 11
129136
runs-on: macOS-11
130137
strategy:
138+
fail-fast: false
131139
matrix:
132-
xcode: ["12.4", "12.5.1", "13.0"]
140+
xcode: ["12.5.1", "13.2.1"]
133141
steps:
134142
- uses: actions/checkout@v2
135143

@@ -166,8 +174,9 @@ jobs:
166174
name: Test without SPM on Linux
167175
runs-on: ubuntu-latest
168176
strategy:
177+
fail-fast: false
169178
matrix:
170-
swift: ["5.3.3", "5.5.2"]
179+
swift: ["5.4.3", "5.5.3"]
171180
steps:
172181
- uses: actions/checkout@v2
173182
- uses: actions/setup-node@v2

.swift-version

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
5.3
1+
5.4

Dockerfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM swift:5.5.2-focal
1+
FROM swift:5.5.3-focal
22

33
MAINTAINER Orta Therox
44

Package.swift

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// swift-tools-version:5.3
1+
// swift-tools-version:5.4
22

33
import PackageDescription
44

@@ -45,7 +45,7 @@ let package = Package(
4545
.target(name: "DangerDependenciesResolver", dependencies: ["DangerShellExecutor", "Version", "Logger"]),
4646
.target(name: "Danger", dependencies: ["OctoKit", "Logger", "DangerShellExecutor"]),
4747
.target(name: "RunnerLib", dependencies: ["Logger", "DangerShellExecutor", "Version"]),
48-
.target(name: "Runner", dependencies: ["RunnerLib", "Logger", "DangerDependenciesResolver"]),
48+
.executableTarget(name: "Runner", dependencies: ["RunnerLib", "Logger", "DangerDependenciesResolver"]),
4949
.target(name: "DangerFixtures", dependencies: ["Danger"]),
5050
] + devTargets
5151
)

Scripts/install.sh

+3-27
Original file line numberDiff line numberDiff line change
@@ -1,45 +1,21 @@
11
#!/bin/bash
22

3-
lower () {
4-
IFS='.' read -a v1_array <<< "$1"
5-
IFS='.' read -a v2_array <<< "$2"
6-
v1=$((v1_array[0] * 100 + v1_array[1] * 10 + v1_array[2]))
7-
v2=$((v2_array[0] * 100 + v2_array[1] * 10 + v2_array[2]))
8-
diff=$((v2 - v1))
9-
10-
((diff > 0)) && return 0
11-
return 1
12-
}
13-
143
TOOL_NAME="danger-swift"
154
PREFIX=${PREFIX:='/usr/local'}
165
INSTALL_PATH="$PREFIX/bin/$TOOL_NAME"
17-
BUILD_PATH=".build/release/$TOOL_NAME"
186
LIB_INSTALL_PATH="$PREFIX/lib/danger"
197
declare -a SWIFT_LIB_FILES=('libDanger.dylib' 'libDanger.so' 'Danger.swiftdoc' 'Danger.swiftmodule' 'OctoKit.swiftdoc' 'OctoKit.swiftmodule' 'RequestKit.swiftdoc' 'RequestKit.swiftmodule' 'Logger.swiftdoc' 'Logger.swiftmodule' 'DangerShellExecutor.swiftdoc' 'DangerShellExecutor.swiftmodule')
208

21-
229
DANGER_LIB_DECLARATION='\.library(name:\ \"Danger\", targets: \[\"Danger\"\])'
2310
DANGER_LIB_DYNAMIC_DECLARATION='\.library(name:\ \"Danger\",\ type:\ \.dynamic,\ targets:\ \[\"Danger\"\])'
24-
sed "s/$DANGER_LIB_DECLARATION/$DANGER_LIB_DYNAMIC_DECLARATION/g" Package.swift > tmpPackage
11+
sed "s/$DANGER_LIB_DECLARATION/$DANGER_LIB_DYNAMIC_DECLARATION/g" Package.swift > tmpPackage
2512
mv -f tmpPackage Package.swift
2613

2714
swift package clean
2815

29-
SWIFT_VERSION=`swift -version | head -n 1 | perl -lpe 's/.*version\ (\d\.\d\.\d).*/$1/'`
30-
31-
UNAME_OUT="$(uname -s)"
32-
33-
lower $SWIFT_VERSION '5.3.0'
34-
35-
if [[ "$?" -eq 0 || "$OSTYPE" == "darwin"* ]]
36-
then
16+
if [[ "$OSTYPE" == "darwin"* ]]; then
3717
BUILD_FOLDER=".build/release"
38-
if [[ `uname -m` == 'arm64' ]]; then
39-
swift build --disable-sandbox --arch arm64 -c release
40-
else
41-
swift build --disable-sandbox -c release
42-
fi
18+
swift build --disable-sandbox -c release
4319
else
4420
BUILD_FOLDER=".build/debug"
4521
swift build --disable-sandbox

0 commit comments

Comments
 (0)