-
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
Introduce "outputDir" property of "codegenConfig" #41782
Conversation
This pull request was exported from Phabricator. Differential Revision: D51494009 |
Base commit: 5d1eac0 |
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
4fe8a62
to
9026e9b
Compare
This pull request was exported from Phabricator. Differential Revision: D51494009 |
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
9026e9b
to
cba3c90
Compare
This pull request was exported from Phabricator. Differential Revision: D51494009 |
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
cba3c90
to
c80b9d0
Compare
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
This pull request was exported from Phabricator. Differential Revision: D51494009 |
c80b9d0
to
4c32c0c
Compare
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
This pull request was exported from Phabricator. Differential Revision: D51494009 |
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
4c32c0c
to
4c9d79d
Compare
This pull request was exported from Phabricator. Differential Revision: D51494009 |
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
4c9d79d
to
b19967d
Compare
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
This pull request was exported from Phabricator. Differential Revision: D51494009 |
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
b19967d
to
1660bcb
Compare
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
This pull request was exported from Phabricator. Differential Revision: D51494009 |
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
Summary: This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009
1660bcb
to
8ff060e
Compare
This pull request was exported from Phabricator. Differential Revision: D51494009 |
This pull request was successfully merged by @dmytrorykun in d45a01d. When will my fix make it into a release? | Upcoming Releases |
Summary: Pull Request resolved: facebook#41782 This diff adds `outputDir` property to `codegenConfig`. Now codegen output dir is resolved like this: 1. It is set to `outputDir` argument of `generate-codegen-artifacts.js` if it is present. 2. *[New]* It is set to `outputDir` property of `codegenConfig` if it is present. 3. It is set to the project root. Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig" Reviewed By: cipolleschi Differential Revision: D51494009 fbshipit-source-id: 0f6e3607b29a3c6d228a88a9460d55bb65c7e55a
Summary:
This diff adds
outputDir
property tocodegenConfig
.Now codegen output dir is resolved like this:
outputDir
argument ofgenerate-codegen-artifacts.js
if it is present.outputDir
property ofcodegenConfig
if it is present.Changelog: [General][Added] - Introduce "outputDir" property of "codegenConfig"
Reviewed By: cipolleschi
Differential Revision: D51494009