-
Notifications
You must be signed in to change notification settings - Fork 24.5k
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 indent for props generation of codegen #24438
Conversation
@zhongwuzw mind updating the snapshots? Looks like they show the changes pretty well :) |
@thymikee Ahha, I'll update. |
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.
Nice!
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.
@cpojer is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
This pull request was successfully merged by @zhongwuzw in 2a4a9f8. When will my fix make it into a release? | Upcoming Releases |
Summary: Part of #24438, add indent foe event emitter header codegen. cc. cpojer rickhanlonii. [General] [Fixed] - Add indent for event emitter header generation of codegen Pull Request resolved: #24445 Differential Revision: D14930352 Pulled By: cpojer fbshipit-source-id: 8bd03b8e12ccf9bd06546acdb744541fcb92eea1
Wow thanks! |
Summary
Add correct indent when run codegen.


Before:
After:


cc. @cpojer @rickhanlonii
Changelog
[General] [Fixed] - Add indent for props generation of codegen
Test Plan
N/A.