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

SPM-compatible project structure. Linux support. Better podfile. CocoaPods deploy action #77

Merged
merged 14 commits into from
Aug 12, 2023
Merged

Conversation

ypopovych
Copy link
Contributor

Hello.

I did a couple of updates to the repo and fixes:

  1. Updated project structure to be more SPM-like
  2. Moved MG code to MGTools target and added it to SPM so it can be run built on Linux
  3. Moved MG Tests to own target
  4. Added tests to the Podfile
  5. Added CocoaPods tests run phase to CI.
  6. Added Linux tests to CI.
  7. Fixed and added tests for: BigInt initializer crashes on large integers #76
  8. Fixed == and != for negative integers and added tests for it.
  9. Added deploy workflow to the CI for CocoaPods repo push.

@mkrd mkrd requested review from twodayslate and mkrd August 9, 2023 10:12
@mkrd
Copy link
Owner

mkrd commented Aug 9, 2023

@ypopovych Hey, thank you so much for your contribution! Looks good on first sight, I will review it in detail later. @twodayslate, I would also appreciate your review :)

@ypopovych
Copy link
Contributor Author

CocoaPods GitHub action workflow will need COCOAPODS_TRUNK_TOKEN secret variable in the repository. How to obtain it: https://fuller.li/posts/automated-cocoapods-releases-with-ci/

@mkrd mkrd merged commit f98947c into mkrd:master Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants