-
Notifications
You must be signed in to change notification settings - Fork 4
Cleanup and repair repo for 2023 #24
base: master
Are you sure you want to change the base?
Conversation
… localization exporting Unit tests pass, integration tests fail
8044897
to
1c3ba01
Compare
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.
Just added some notes about some non-obvious things I did to cleanup the repo. Most were around addressed warnings specifically from XCode, and changing Build Settings were done through the GUI in XCode.
@@ -166,7 +166,7 @@ class SpreedlyClientImpl: NSObject, SpreedlyClient { | |||
"platform": "apple", | |||
"locale": Locale.current.languageCode ?? "unknown", | |||
"os": [ | |||
"name": proc.operatingSystemName(), | |||
"name": proc.operatingSystemVersionString, |
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 was replaced since operatingSystemName()
is now deprecated, and Apple recommends using -operatingSystemVersionString
instead. But this also means that both fields name
and version
have the same value, so this might need addressing later on.
Src: https://developer.apple.com/documentation/foundation/processinfo/1407388-operatingsystemname
<key>IDEDidComputeMac32BitWarning</key> | ||
<true/> | ||
</dict> | ||
</plist> |
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.
Committing this to the repo was recommended by Apple.
"$(inherited)", | ||
"$(PROJECT_DIR)/Frameworks", | ||
); | ||
FRAMEWORK_SEARCH_PATHS = "$(inherited)"; |
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.
Deleting outdated framework search paths was recommended as a fix for the ld: warning: directory not found for option '-F...
warning.
Src: https://stackoverflow.com/questions/9458739/ld-warning-directory-not-found-for-option
@@ -801,7 +800,6 @@ | |||
F076C6D90DB36BC555C246AC /* BankAccountInfo.swift in Sources */, | |||
A60A0240278796000044BD69 /* SpreedlyVersion.swift in Sources */, | |||
F076CD7FF9E8D8C63D808B1A /* CreditCardInfo.swift in Sources */, | |||
442A2D2E48133A09DBBD9299 /* Package.swift.template in Sources */, |
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.
Removed this non-Swift file from the list of Compile Sources to fix the no rule to process file...
warning for this file.
Wanted to note that I suspect that the Apple Pay integration tests fail because of outdated Apple Pay payment method data, where the year is behind, since the response from Spreedly is: {"errors":[{"attribute":"year","key":"errors.invalid","message":"Year is invalid"}]} This causes the |
This is currently a work in progress to address tech debt and update the project.