search

mindbox-cloud / ios-sdk

Other
6 stars 16 forks source link

MBX-0000: Xcode recommendations fix #452

Closed justSmK closed 1 month ago

justSmK commented 1 month ago

Здесь можно посмотреть какие пункты Xcode предлагал исправить конкретно в настройках проекта

Apple Module Verifier

https://developer.apple.com/documentation/xcode/identifying-and-addressing-framework-module-issues

Always Embed Swift Standard Libraries

Протестировал CocoaPods и SPM с обновлениями, всё было хорошо.

Всё разбито строго по коммитам

justSmK commented 1 month ago

Нужно обратить внимание на подход с отключением deprecated warnings. Возможно в тестах его стоит оставить, а в SDK убрать, или везде использоваться. Или забить и смириться конкретно с этим типом предупреждений.

justSmK commented 1 month ago

MBX-0000: MindboxTests fix InAppTargetingRequestsTests with deprecated

Тут полностью можно убрать использование deprecated переменной в принципе, судя по тестам и реализации основного класса.

Надо сделать мне будет фикс и закомменченный вариант выкинуть, если апрувнем это