Skip to content
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 SchemaLayout & OperationsLayout #5573

Merged
merged 2 commits into from
Jan 29, 2024
Merged

Conversation

martinbonnin
Copy link
Contributor

@martinbonnin martinbonnin commented Jan 29, 2024

(Apologies for the initial title, clicked "submit" a bit too early)

Introduce SchemaLayout and OperationsLayout that are specialized versions so that someone building the server code doesn't need to specify operations/fragments package names.

Also remove default values from options constructors. Given the number of options, it's way too easy to forget passing some. Instead, add buildFoo() functions that can be used for users of the ApolloCompiler API

…he Gradle plugin, use the primary constructor

introduce SchemaLayout & OperationsLayout
Copy link

netlify bot commented Jan 29, 2024

Deploy Preview for apollo-android-docs canceled.

Name Link
🔨 Latest commit ca45118
🔍 Latest deploy log https://app.netlify.com/sites/apollo-android-docs/deploys/65b7bce13399700008bee410

@martinbonnin martinbonnin changed the title add buildCodegenOptions() for times when we want default values. In the Gradle plugin, use the primary constructor introduce SchemaLayout & OperationsLayout Introduce SchemaLayout & OperationsLayout Jan 29, 2024
Copy link
Contributor

@BoD BoD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

…ollo3/compiler/Options.kt

Co-authored-by: Benoit 'BoD' Lubek <BoD@JRAF.org>
@martinbonnin martinbonnin merged commit 58fc2e4 into main Jan 29, 2024
9 checks passed
@martinbonnin martinbonnin deleted the 01-schema-layout branch January 29, 2024 16:18
martinbonnin added a commit that referenced this pull request Feb 5, 2024
* add buildCodegenOptions() for times when we want default values. In the Gradle plugin, use the primary constructor
introduce SchemaLayout & OperationsLayout

* Update libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo3/compiler/Options.kt

Co-authored-by: Benoit 'BoD' Lubek <BoD@JRAF.org>

---------

Co-authored-by: Benoit 'BoD' Lubek <BoD@JRAF.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants