-
-
Notifications
You must be signed in to change notification settings - Fork 114
Upstream/985 combine multiple response projections #1032
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
Upstream/985 combine multiple response projections #1032
Conversation
|
In order to verify major changes, I put Scala and kotlin files into the test module to ensure that they can be compiled, but at present, I need to manually execute GraphQLCodegenVerifyCompileTest.java |
| void cleanup() { | ||
| Utils.deleteDir(outputBuildDir); | ||
| } | ||
| // @AfterEach |
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.
@jxnu-liguobin can you please revert this
| id "signing" | ||
| id "maven-publish" | ||
| id "org.sonarqube" version "3.2.0" | ||
| id 'org.jetbrains.kotlin.jvm' version '1.8.20-Beta' |
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.
@jxnu-liguobin why are we adding this?
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.
@jxnu-liguobin I would like not to add kotlin code and dependencies to the main project. instead, please add this as part of example kotlin plugin which we already have.
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.
Yes. We can update the sample synchronously when major changes occur.
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.
The template changes in this PR have been verified, closed it
Description
Related to #ISSUE_NUMBER
Changes were made to: