CombineX icon indicating copy to clipboard operation
CombineX copied to clipboard

Published 20 hours ago • verified publisher icon cx-org

(fix #52)Add integration tests for package managers

Open luoxiu opened this issue 3 years ago • 5 comments

luoxiu avatar Jun 01 '21 00:06 luoxiu

Would you please remove latest swift (failing tests) for now? so we can get clean ci result. Also add a # FIXME mark, I'll look into it later.

ddddxxx avatar Jun 01 '21 03:06 ddddxxx

🤔 https://github.com/cx-org/CombineX/runs/2723742441?check_suite_focus=true#step:3:396

luoxiu avatar Jun 02 '21 01:06 luoxiu

Still here? Better use relative path or ignore it.

Will add this to gitignore, 🤦🏻‍♀️, carthage doesn't support relative path as far as I know.

luoxiu avatar Jun 02 '21 09:06 luoxiu

What is the purpose of the extra folder package-manager in IntegrationTests/? I prefer a flatter structure. Probably IntegrationTests/[SwiftPackage|Carthage|CocoaPods]/?

Will add more integration tests in the future, for example, test the integration with CxCocoa

luoxiu avatar Jun 02 '21 09:06 luoxiu

for example, test the integration with CxCocoa

Does such test belong to CXCocoa?

ddddxxx avatar Jun 02 '21 10:06 ddddxxx