-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update CircleCI config #138
base: master
Are you sure you want to change the base?
Conversation
@@ -2,7 +2,8 @@ version: 2.1 | |||
jobs: | |||
build-and-test: | |||
macos: | |||
xcode: 15.0.0 | |||
xcode: 15.0.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be specified less specific? Having to update for each patch version seems unnecessary. What about the resource_class, can that be more general?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was trying to check in their docs, but found no example of how to be less specific, but could of course just try to say 15 and see what happens 🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works for Travis to use just 15 or 15.0, we should try same here...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works for Travis to use just 15 or 15.0, we should try same here...
They are two completely different services though..
Was trying to check in their docs, but found no example of how to be less specific, but could of course just try to say 15 and see what happens 🤷
They specify 15.0.0
in their docs https://circleci.com/docs/testing-ios/ but might just default to 15
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For sure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately 15 didn't work so changing back to 15.0.1
Why?
It was a little bit old.
What?
Version Change
No changes to the library