-
Notifications
You must be signed in to change notification settings - Fork 664
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
Merge main in dev-3.x #3580
Merge main in dev-3.x #3580
Conversation
* bump Kotlin and Coroutines to 1.5.0 * make the KotlinCompile task depend on antlr generation * update metalava-plugin * update metalava * update metalava using Jdk8. Not really sure why the signatures would differ based on the java version...
* Implement Query Batching for `apollo-runtime` * update metalava signatures * Trigger HTTP call on max batch size, added synchronization, throw when using GET method * Unit tests for BatchPoller * Fix checkstyle * Propagate parsing errors in callbacks, better syncrhonization, use LinkedList * Added some tests for batch response parsing * Added tests for request body + headers * Builder API to enable per query batching, dont rely on time for tests * update Metalava signatures Co-authored-by: Martin Bonnin <martin@mbonnin.net>
* feat: add Reactor support resolves #627 * remove .gitignore that is handled by the top level one * update metalava signatures Co-authored-by: Martin Bonnin <martin@mbonnin.net>
update dep version forcing as per gradle new api
inspired by commit 2e2f5d9
* Configure for updated docs theme * Upgrade docs theme * Bump docs theme version * Bump docs theme version * Update docs theme version * Move config from docs netlify.toml to root level netlify.toml * Upgrade docs theme * Upgrade docs theme * Upgrade docs theme * Upgrade docs theme * Bump docs theme to v5 Co-authored-by: Trevor Blades <tdblades@gmail.com>
@@ -41,7 +41,7 @@ dependencies { | |||
testImplementation(groovy.util.Eval.x(project, "x.dep.okHttp.mockWebServer4")) | |||
} | |||
|
|||
if (true) { |
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.
Pushed by mistake?
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.
👏
And... I forgot to change the button to "merge" and squashed everything instead 🤦 . I pushed a merge manually there so that we have a reference point: aa52f01b3494f765bac8216601c46d7e1 |
A few noteworthy items:
Closes #3574
I don't know why this shows -29k lines at the moment, the diff is nowhere that huge 🤷It's package-lock.json