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

Say Goodbye to Apollo CLI #2003

Merged
merged 6 commits into from
Oct 29, 2021
Merged

Conversation

calvincestari
Copy link
Member

@calvincestari calvincestari commented Oct 26, 2021

Closes #2001

This PR is branched off of PR #1999 but I do not intend to merge it into that branch. Once the parent PR (#1999) is merged GitHub should automagically change the base branch to release/1.0-alpha-incubating which is the intended target.

On to the business of this PR:

  • Removes the Apollo CLI from codegen. I believe I've caught all related code too; extensions, tests, helpers, etc. however if you know of something I've missed please let me know.
  • Does not include any changes to the Swift scripts; those haven't caught up with the recent changes to ApolloCodegenConfiguration anyways so I'll put that into a different PR, together with a couple other related bits.

Goodbye CLI, it's been grand, you can see yourself out now - Totsiens, Au Revoir, Auf Wiedersehen, Yasou, Aloha, Namaste, Arrivederci, Sayōnara, Adios, Hamba kahle, Do pobachennia!

@@ -1468,6 +1437,8 @@
children = (
9BAEEBEB234663F200808306 /* ApolloSchemaDownloader.swift */,
9BAEEBED2346644600808306 /* ApolloSchemaDownloadConfiguration.swift */,
9BCA8C0826618226004FF2F6 /* UntypedGraphQLRequestBodyCreator.swift */,
Copy link
Member Author

Choose a reason for hiding this comment

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

There two files are moved into the SchemaDownload Xcode folder - it's the only place they're needed.

Copy link
Contributor

@AnthonyMDev AnthonyMDev left a comment

Choose a reason for hiding this comment

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

YESSSSSSS! 🔥

Copy link
Contributor

@designatednerd designatednerd left a comment

Choose a reason for hiding this comment

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

One comment about tests otherwise:
queen elizabeth saying "Bye felicia"

@calvincestari calvincestari added this to the Release 1.0 milestone Oct 26, 2021
@calvincestari calvincestari added the codegen Issues related to or arising from code generation label Oct 26, 2021
@calvincestari
Copy link
Member Author

Once #1999 is merged I'll fix the merge conflicts. It's a good blocker on this for now to prevent accidental merging.

Base automatically changed from 1.0/refactor-filemanager-extension to release/1.0-alpha-incubating October 29, 2021 21:04
@calvincestari
Copy link
Member Author

This is now rebased and will be merged ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codegen Issues related to or arising from code generation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants