search

minvws / nl-covid19-notification-app-ios

European Union Public License 1.2
120 stars 36 forks source link

CI: Use the default Xcode version #68

Closed EwoutH closed 3 years ago

EwoutH commented 3 years ago

Ensures an up-to-date version of Xcode is used.

The currently default version can be found here: https://github.com/actions/virtual-environments/blob/main/images/macos/macos-10.15-Readme.md#xcode

roelspruit commented 3 years ago

Hi. This seems like a good idea in principal but we might have an issue with automatically updating to the latest Xcode. We are likely to experience snapshot test errors between different Xcode versions so switching automatically might lead to sudden, unexpected failures. Internally our (azure) pipeline does already use the current default xcode (12.3) so I'm fine with switching to 12.3 on GitHub too, but it will have to remain manually set for now. This is of course something we should remember to do ourselves when we switch the internal pipeline.

EwoutH commented 3 years ago

Thanks for the explanation.