Skip to content
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

Task: Look into using dynamic lib instead of static lib #30

Closed
1 task
surpher opened this issue Sep 15, 2020 · 0 comments · Fixed by #41
Closed
1 task

Task: Look into using dynamic lib instead of static lib #30

surpher opened this issue Sep 15, 2020 · 0 comments · Fixed by #41
Labels
help wanted Extra attention is needed tech Something technical that doesn't fit a feature or enhancement

Comments

@surpher
Copy link
Owner

surpher commented Sep 15, 2020

❕ Problem Statement

The MockServer used in this framework is built from a Rust codebase and is built into a static lib (.a). This means we end up with huge lib files (eg: the .a file for iOS is more than 100MB because it includes a slice for iOS simulator and iOS device). This means a lot of traffic on initial and subsequent MockServer updates pulls. That also means that the updated .a lives in GitHub LFS and each pull/push, and triggered GitHub Action eats into the Storage and Bandwidth quota. Far from ideal.

💬 Task Description

Because PactSwift should not be included in the App Store build, consider preparing dynamic libs to be used with test target.

It may require a different project setup at user end (for the ones already using it), but if we can reduce our size as a dependency, that's a good trade-off.

👩‍🔧 Technical Design Notes

  • Check if we can replace macOS static lib with a dynamic lib

  • Replacing iOS binary could be a separate concern

🤝 Relationships

@surpher surpher added help wanted Extra attention is needed tech Something technical that doesn't fit a feature or enhancement labels Sep 15, 2020
@surpher surpher added this to To do in Beta release - consumer via automation Sep 30, 2020
Beta release - consumer automation moved this from To do to Done Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed tech Something technical that doesn't fit a feature or enhancement
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant