-
Notifications
You must be signed in to change notification settings - Fork 253
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
Update deno, so we can generate docs again #3626
Conversation
This comment has been minimized.
This comment has been minimized.
CI performance tests
|
Condition: | ||
condition: sendValue | ||
if_clause: | ||
Flatten: |
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.
@Geal I've tagged you as reviewer since this needs attention and i /think/ you worked on it for the auth pr
@@ -22,6 +22,7 @@ To compile the Apollo Router, you need to have the following installed: | |||
<!-- renovate-automation: rustc version --> | |||
* [Rust 1.71.1 or later](https://www.rust-lang.org/tools/install) | |||
* [Node.js 16.9.1 or later](https://nodejs.org/en/download/) | |||
* [CMake 3.5.1 or later](https://cmake.org/download/) |
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.
Should we add protoc to the prerequisites?
@@ -22,6 +22,7 @@ To compile the Apollo Router, you need to have the following installed: | |||
<!-- renovate-automation: rustc version --> | |||
* [Rust 1.71.1 or later](https://www.rust-lang.org/tools/install) | |||
* [Node.js 16.9.1 or later](https://nodejs.org/en/download/) | |||
* [CMake 3.5.1 or later](https://cmake.org/download/) |
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.
@o0Ignition0o Should we retroactively add something to CHANGELOG saying that as of v1.29.0, building Router from source requires CMake to be installed?
Fix #3305, #3684
Router docs failed to build on crates.io because of a documentation compile error in Deno.
This updates Deno to the latest version, which allows us to generate crates.io documentation again.