-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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 duplicated code that resulted after a merge conflict. #37477
Conversation
This pull request was exported from Phabricator. Differential Revision: D45979013 |
Base commit: 6d24ee1 |
…k#37477) Summary: Pull Request resolved: facebook#37477 When creating a stack of several codegen fixes, a conflit generated some duplicated code we moved to the parsers from the utils. This change removes that duplicated cone. ## Changelog: [Genearal][Fixed] - Remove duplicated code. Reviewed By: cortinico Differential Revision: D45979013 fbshipit-source-id: 4195bab3ab7e27a56cbec92cdb032b5a3d7b1b73
c221d1c
to
cf1954e
Compare
This pull request was exported from Phabricator. Differential Revision: D45979013 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D45979013 |
…k#37477) Summary: Pull Request resolved: facebook#37477 When creating a stack of several codegen fixes, a conflit generated some duplicated code we moved to the parsers from the utils. This change removes that duplicated cone. ## Changelog: [Genearal][Fixed] - Remove duplicated code. Reviewed By: cortinico Differential Revision: D45979013 fbshipit-source-id: 7a9dde2c9c0dfeb24f45d6a5f96c384ad1423108
cf1954e
to
aa80a45
Compare
…k#37477) Summary: Pull Request resolved: facebook#37477 When creating a stack of several codegen fixes, a conflit generated some duplicated code we moved to the parsers from the utils. This change removes that duplicated cone. ## Changelog: [Genearal][Fixed] - Remove duplicated code. Reviewed By: cortinico Differential Revision: D45979013 fbshipit-source-id: c23dbb491c5ed2be5e29ae2719851772cc1f75c8
aa80a45
to
aeef68e
Compare
This pull request was exported from Phabricator. Differential Revision: D45979013 |
Summary: We recently landed a change with a typo in the interface declaration. We just fixed the typo ## CHANGELOG [General][Fixed] - Fixed a typo in the interface declaration Differential Revision: D45978419 fbshipit-source-id: da6ab32273a01c8c42750f6cc736a8c5bd11510f
…k#37477) Summary: Pull Request resolved: facebook#37477 When creating a stack of several codegen fixes, a conflit generated some duplicated code we moved to the parsers from the utils. This change removes that duplicated cone. ## Changelog: [Genearal][Fixed] - Remove duplicated code. Reviewed By: cortinico Differential Revision: D45979013 fbshipit-source-id: 4f75089af1e2e9cf4f2ee806953ce407aa0408dd
aeef68e
to
905b6a0
Compare
This pull request was exported from Phabricator. Differential Revision: D45979013 |
This pull request has been merged in 8dcaa4c. |
Summary:
When creating a stack of several codegen fixes, a conflit generated some duplicated code we moved to the parsers from the utils.
This change removes that duplicated cone.
Changelog:
[Genearal][Fixed] - Remove duplicated code.
Differential Revision: D45979013