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

Make apollo-router compile on docs.rs build servers #1847

Merged
merged 2 commits into from
Sep 21, 2022
Merged

Conversation

SimonSapin
Copy link
Contributor

apollographql/federation-rs#185 was not sufficient since each package being published to crates.io needs to have its own [package.metadata.docs.rs] config.

apollographql/federation-rs#185 was not sufficient
since each package being published to crates.io needs to have its own
`[package.metadata.docs.rs]` config.
@SimonSapin SimonSapin self-assigned this Sep 20, 2022
Copy link
Contributor

@o0Ignition0o o0Ignition0o left a comment

Choose a reason for hiding this comment

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

wow great catch!

@SimonSapin SimonSapin added this to the v1.0.0 milestone Sep 20, 2022
A new test checks that it is up to date.
@SimonSapin
Copy link
Contributor Author

I tried this on docs.rs which uncovered another issue: https://docs.rs/crate/lets-see-if-this-builds-on-docs-rs/0.0.1/builds/633305

The new commit attempts to fix that, with https://docs.rs/crate/lets-see-if-this-builds-on-docs-rs/0.0.2 currently in the build queue.

@SimonSapin SimonSapin changed the title Disable Deno snapshotting in router-bridge when docs.rs builds docs Make apollo-router compile on docs.rs build servers Sep 20, 2022
@SimonSapin
Copy link
Contributor Author

https://docs.rs/lets-see-if-this-builds-on-docs-rs/0.0.2/lets_see_if_this_builds_on_docs_rs/ builds!

@abernix abernix merged commit 0ffc1d2 into main Sep 21, 2022
@abernix abernix deleted the simon/docs-rs branch September 21, 2022 07:54
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.

4 participants