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

Remove float cast flag in 0.10.0 branch #654

Merged
merged 1 commit into from
Jan 5, 2022

Conversation

neildhar
Copy link
Contributor

No description provided.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Dec 13, 2021
@mrousavy
Copy link
Contributor

mrousavy commented Dec 16, 2021

Don't have permission to approve this PR - but my build now got accepted by Apple with this PR! 👍

So this PR fixes #649 🚀

Summary:
This breaks the open source iOS build, back it out temporarily since it
doesn't have the intended effect anyway.

Reviewed By: mhorowitz

Differential Revision: D33290406

fbshipit-source-id: b5dbfa93a1a04e1e1b53d48b857cba867393eaaf
@neildhar neildhar changed the title Remove float cast flag Remove float cast flag in 0.10.0 branch Jan 5, 2022
@neildhar neildhar marked this pull request as ready for review January 5, 2022 06:22
@neildhar neildhar merged commit d57a7f8 into facebook:release-v0.10 Jan 5, 2022
@neildhar neildhar deleted the remove-float-cast-flag branch January 5, 2022 10:36
@kelset
Copy link

kelset commented Jan 12, 2022

has this changed also been replicated into main, btw?

@neildhar
Copy link
Contributor Author

@kelset Yes, it is in main as 20aa6fa

@kelset
Copy link

kelset commented Jan 12, 2022

cool - so it's either a matter of making a 0.10.1 or a 0.11, if I'm understanding the current status of releases for Hermes.

Considering that for 0.67 we ended up staying on 0.9, maybe at this point might be worth just coordinating the branch cut for 0.68 with a 0.11 for hermes?
(cc @lunaleaps)

@neildhar
Copy link
Contributor Author

I was actually wondering whether we should just replace the artefact for the 0.10.0 CocoaPod with the patched version, given that this is a minor but necessary change for it to work. But I am currently away and haven't had a chance to think through and discuss any potential pitfalls of that approach.

@billnbell
Copy link

??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants