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

fix(ios): Fix Unsupported swift architecture #393

Merged
merged 14 commits into from
Sep 15, 2024

Conversation

AbdullahFaqeir
Copy link
Contributor

@AbdullahFaqeir AbdullahFaqeir commented Sep 14, 2024

Fix when compiling module
Needed for #391

FINALY after around 7 hoursl

@AbdullahFaqeir AbdullahFaqeir changed the title Fix Unsupported swift architecture fix(ios): Fix Unsupported swift architecture Sep 14, 2024
@AbdullahFaqeir
Copy link
Contributor Author

AbdullahFaqeir commented Sep 14, 2024

@m1ga @hansemannn This was a good day for me, if e can approve the work flows for everything as I can't wait honestly

@m1ga
Copy link
Contributor

m1ga commented Sep 14, 2024

@AbdullahFaqeir nice! Can you please add the .github/actions from https://github.com/tidev/hyperloop.next/pull/391/files so we can see that it is building online

@AbdullahFaqeir
Copy link
Contributor Author

@AbdullahFaqeir nice! Can you please add the .github/actions from https://[email protected]/tidev/hyperloop.next/pull/391/files so we can see that it is building online

@m1ga done

@m1ga
Copy link
Contributor

m1ga commented Sep 14, 2024

@AbdullahFaqeir I don't see the files in this PR. Can you check it again please

@AbdullahFaqeir
Copy link
Contributor Author

@m1ga
I'm really sorry but I'm kinda noop when it comes to github actions

@m1ga
Copy link
Contributor

m1ga commented Sep 14, 2024

no problem! We are almost there 😄 I can push some stuff later so minimize the changes. Happy that you help here to solve the iOS issue!

@AbdullahFaqeir
Copy link
Contributor Author

When can we have it running, I'm really excited, I want to move to the next one hahaah

@AbdullahFaqeir
Copy link
Contributor Author

@m1ga @hansemannn can someone send me the CLA

And I'm proud I fixed it, proud that I'm giving back to you guys, proud of every hour I spent on each PR

@m1ga m1ga self-requested a review September 14, 2024 22:51
Copy link
Contributor

@m1ga m1ga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Both Android and iOS actions build now 👍 The CLA check is failing because the first commit was not done with the Github user.

@hansemannn please check it and then you can merge this and after that #391

Then we can create a new release to use it in the gradle 8 PR 👍

@hansemannn hansemannn merged commit d727b18 into tidev:master Sep 15, 2024
2 of 3 checks passed
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