-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Replace folly::Optional with std::optional #35436
Conversation
This pull request was exported from Phabricator. Differential Revision: D41415031 |
Base commit: a672869 |
Base commit: a672869 |
PR build artifact for 372f96e is ready. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome stuff! I've checked the CMake part and the code looks generally fine to me 👍
@@ -17,7 +17,7 @@ endif(CCACHE_FOUND) | |||
|
|||
# Make sure every shared lib includes a .note.gnu.build-id header | |||
add_link_options(-Wl,--build-id) | |||
add_compile_options(-Wall -Werror -std=c++1y) | |||
add_compile_options(-Wall -Werror -std=c++17) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this
This pull request was exported from Phabricator. Differential Revision: D41415031 |
Summary: Pull Request resolved: facebook#35436 Using std::optional as react-native has been using C++17 for quite some time changelog: [Internal] Reviewed By: cortinico Differential Revision: D41415031 fbshipit-source-id: ea27ff45a0f595098c7a974e507b8493fbb33ebb
372f96e
to
d79d435
Compare
PR build artifact for d79d435 is ready. |
Summary: Pull Request resolved: facebook#35436 Using std::optional as react-native has been using C++17 for quite some time changelog: [Internal] Reviewed By: cortinico Differential Revision: D41415031 fbshipit-source-id: 8165a4f1b24c49c58fcb5af59a50f31e96816cc4
This pull request was exported from Phabricator. Differential Revision: D41415031 |
d79d435
to
0c1ca87
Compare
PR build artifact for 0c1ca87 is ready. |
This pull request was successfully merged by @christophpurrer in 022e22c. When will my fix make it into a release? | Upcoming Releases |
Summary: Pull Request resolved: facebook#35436 Using std::optional as react-native has been using C++17 for quite some time changelog: [Internal] Reviewed By: cortinico Differential Revision: D41415031 fbshipit-source-id: d786647f64b4f90cf75409109830ae0885460c17
Summary:
Using std::optional as react-native has been using C++17 for quite some time
changelog: [Internal]
Differential Revision: D41415031