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

Compile for Android #138

Merged
merged 1 commit into from
Nov 12, 2024
Merged

Conversation

johankool
Copy link
Contributor

No description provided.

@mbrandonw
Copy link
Member

Hi @johankool, thanks for looking into this! Are you familiar with how we can get CI testing for android so that we can prevent problems like this in the future?

@johankool
Copy link
Contributor Author

I used the Android SDK from here: https://github.com/finagolfin/swift-android-sdk then ran the .xctest on an actual device. The linked project has a workflow that shows how you can use the SDK and run tests on an emulator as they do that for amongst others the swift-argument-parser framework too.

@johankool
Copy link
Contributor Author

johankool commented Nov 3, 2024

BTW, quite amazing how little effort it takes to get the PointFree libraries to compile for Android. The below I have working already locally. PRs for those are depending on getting the fixes released for swift-issue-reporting/xctest-dynamic-overlay and swift-concurrency-extras.

  • swift-case-paths
  • swift-clocks
  • swift-concurrency-extras
  • swift-custom-dump
  • swift-html
  • swift-identified-collections
  • swift-macro-testing
  • swift-parsing
  • swift-snapshot-testing

@johankool
Copy link
Contributor Author

Now that this PR was merged by @stephencelis, it'd be great if this one could be merged too. That way I can prepare the PRs for the other projects as well to get them to work on Android.

@stephencelis
Copy link
Member

Thanks!

@stephencelis stephencelis merged commit 05c9f02 into pointfreeco:main Nov 12, 2024
17 checks passed
@johankool johankool deleted the android-support branch November 12, 2024 07:39
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.

3 participants