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

Add Apple Silicon targets #201

Merged
merged 6 commits into from
Oct 21, 2021

Conversation

ychescale9
Copy link
Contributor

@ychescale9 ychescale9 commented Sep 2, 2021

Fixes #199.

Temporarily usingcom.freeletics.mad:state-machine:0.3.0-SNAPSHOT (testing locally) until this merged and released.

Remaining dependencies missing apple silicon targets:

  • kotest
  • turbine

@sockeqwe
Copy link
Collaborator

Hi @ychescale9
it looks like kotest now has support for the new apple targets. Could you please update your PR and also merge latest FlowRedux main branch.

Then we should be good to go here with this PR.

@ychescale9
Copy link
Contributor Author

@sockeqwe It's only available in the 5.0.0 milestone release, but if you don't mind we can update to 5.0.0.M2.

@ychescale9 ychescale9 marked this pull request as ready for review October 17, 2021 23:26
@sockeqwe
Copy link
Collaborator

@ychescale9 you are right, I overlookked that it is only available in 5.0.0 releases. Then lets keep this PR open for a bit longer and merge it in the next weeks.

@gabrielittner
Copy link
Member

I think we could also remove kotest. So far we're not using it anywhere

@sockeqwe
Copy link
Collaborator

Very true 🙈 I didnt realizeit before ...

Here is another PR to remove this dependency
#224

@sockeqwe sockeqwe enabled auto-merge (squash) October 21, 2021 10:45
@sockeqwe sockeqwe merged commit 41ce86f into freeletics:main Oct 21, 2021
@sockeqwe
Copy link
Collaborator

Thanks @ychescale9 🎉

@ychescale9 ychescale9 deleted the yc/add-apple-silicon-targets branch October 21, 2021 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Add targets for Apple Silicon
3 participants