search

net-a-porter-mobile / XCTest-Gherkin

Gherkin syntax for XCTestCase
Apache License 2.0
254 stars 64 forks source link

Update travis to include Xcode12 / iOS 14 #172

Closed deanWombourne closed 4 years ago

deanWombourne commented 4 years ago

@kerrmarin Now all i have to do is work out what's going on with travis :(

kerrmarin commented 4 years ago

@deanWombourne:

    - WARN  | [XCTest-Gherkin/Core,XCTest-Gherkin/Native] xcodebuild:  /Users/travis/build/net-a-porter-mobile/XCTest-Gherkin/Pod/Core/XCTestCase+Gherkin.swift:67:14: warning: 'recordFailure(withDescription:inFile:atLine:expected:)' is deprecated: Replaced by 'record(_:)'

    - WARN  | [XCTest-Gherkin/Core,XCTest-Gherkin/Native] xcodebuild:  /Users/travis/build/net-a-porter-mobile/XCTest-Gherkin/Pod/Core/XCTestCase+Gherkin.swift:69:18: warning: 'recordFailure(withDescription:inFile:atLine:expected:)' is deprecated: Replaced by 'record(_:)'
deanWombourne commented 4 years ago

Yea, getting my head round the replacement method.

deanWombourne commented 4 years ago

So, replacement method is only in Xcode 14, but I can't see a way to check which xcode is running. Sigh. --allow-warnings it is :)

kerrmarin commented 4 years ago

😂

deanWombourne commented 4 years ago

Sigh close/open to poke travis to recheck the pr merge commit.