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

release: 3.16.2 #6735

Merged
merged 6 commits into from
Nov 20, 2024
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: missing headers on new arch (#6725)
## Summary
PR adding back headers removed in
#6537.
For some reason, locally they are not needed, probably to some different
setup or having the libraries set globally somewhere, but it caused
failures in Expensify:
https://github.com/Expensify/App/actions/runs/11857735322/job/33046828856.
They should be included since methods from them are used.
## Test plan
Not really, just run CIs probably.
  • Loading branch information
WoLewicki authored and tjzel committed Nov 20, 2024
commit 84b1e7c2738f5162c0337f1141a3786ef5a8cbd3
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@

#ifdef RCT_NEW_ARCH_ENABLED
#include <iomanip>
#include <sstream>
#include <string>
#endif // RCT_NEW_ARCH_ENABLED

// Standard `__cplusplus` macro reference:
Expand Down