-
Notifications
You must be signed in to change notification settings - Fork 61
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
fix: Updates Swift Package.resolved
file version format
#304
Conversation
✅ Deploy Preview for apollo-ios-docc canceled.
|
✅ Deploy Preview for eclectic-pie-88a2ba canceled.
|
} | ||
|
||
func test__matchCLIVersionToApolloVersion__givenPackageResolvedFileInXcodeWorkspaceAndProject_withVersion2FileFormat_hasMatchingVersion_returns_versionMatch_fromWorkspace() throws { | ||
func test__matchCLIVersionToApolloVersion__givenPackageResolvedFileInXcodeWorkspaceAndProject_withKnownResolvedFileFormats_hasMatchingVersion_returns_versionMatch_fromWorkspace() throws { |
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.
This test was changed to have both the workspace and project use the same Apollo version. Previously they were different and that could also affect the outcome. So this test was changed and there is a new test below that has mismatched versions to ensure that the workspace one is still selected.
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.
Thanks for this!
117e41c3 fix: Updates Swift file version format (#304) git-subtree-dir: apollo-ios-codegen git-subtree-split: 117e41c33393b6abae6842f8a0fbe498611668ca
… format git-subtree-dir: apollo-ios-codegen git-subtree-mainline: 1ff20e5 git-subtree-split: 117e41c33393b6abae6842f8a0fbe498611668ca
Fixes apollographql/apollo-ios#3355.
This is the fix to accept version
3
as a valid format for the version checker.