Skip to content

Commit 9418af4

Browse files
Fix building with Swift 5.2 (#91)
* Test both Swift 5.2 and 5.3 * Add missing `self` * Bump checkout versions
1 parent 89bc2c5 commit 9418af4

2 files changed

Lines changed: 13 additions & 14 deletions

File tree

.github/workflows/test.yml

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,16 @@ name: test
22
on:
33
- pull_request
44
jobs:
5-
queues_xenial:
6-
container:
7-
image: swift:5.3-xenial
5+
linux:
86
runs-on: ubuntu-latest
7+
strategy:
8+
matrix:
9+
image:
10+
- swift:5.2-xenial
11+
- swift:5.2-bionic
12+
- swift:5.3-xenial
13+
- swift:5.3-bionic
14+
container: ${{ matrix.image }}
915
steps:
10-
- uses: actions/checkout@v1
11-
- run: swift test --enable-test-discovery --sanitize=thread
12-
queues_bionic:
13-
container:
14-
image: swift:5.3-bionic
15-
runs-on: ubuntu-latest
16-
steps:
17-
- uses: actions/checkout@v1
16+
- uses: actions/checkout@v2
1817
- run: swift test --enable-test-discovery --sanitize=thread

Sources/Queues/QueueWorker.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,12 @@ public struct QueueWorker {
1818
return self.queue.pop().flatMap { id in
1919
//No job found, go to the next iteration
2020
guard let id = id else {
21-
queue.logger.trace("Did not receive ID from pop")
21+
self.queue.logger.trace("Did not receive ID from pop")
2222
return self.queue.eventLoop.makeSucceededFuture(())
2323
}
2424

25-
queue.logger.trace("Received job \(id)")
26-
queue.logger.trace("Getting data for job \(id)")
25+
self.queue.logger.trace("Received job \(id)")
26+
self.queue.logger.trace("Getting data for job \(id)")
2727

2828
return self.queue.get(id).flatMap { data in
2929
var logger = self.queue.logger

0 commit comments

Comments
 (0)