-
Notifications
You must be signed in to change notification settings - Fork 99
Android CI #135
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
base: main
Are you sure you want to change the base?
Android CI #135
Conversation
|
Looks like this wasm step is quite old (Swift 5.9.2). You could try updating based on our most up-to-date library: https://github.com/pointfreeco/swift-sharing/blob/10ba53dd428aed9fc4a1543d3271860a6d4b8dd2/.github/workflows/ci.yml#L44-L60 Or we can look into it soon. |
|
@johankool Sorry for the delay here! I've rebased things and tried to address a few issues, but I'm a little stumped with the current failure. Is it possible swift-issue-reporting has some Android compatibility issues? |
According to the Swift Package Index the main branch works, but 1.7.0 not? https://swiftpackageindex.com/pointfreeco/swift-issue-reporting/builds It all doesn't make much sense to me either. I'll have a poke around this weekend. |
|
Thanks! And no rush, if/whenever you have time 😄 |
Simplify Android CI by using a GitHub action instead.
Ran successfully here: https://github.com/johankool/swift-custom-dump/actions/runs/13340335125
Except wasm fails, but that's not due to these changes.