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

Release docs #389

Merged
merged 13 commits into from
Mar 30, 2021
Merged

Release docs #389

merged 13 commits into from
Mar 30, 2021

Conversation

JakeDawkins
Copy link
Contributor

@JakeDawkins JakeDawkins commented Mar 29, 2021

Todo:

  • graph introspect
  • subgraph introspect
  • supergraph compose

@JakeDawkins JakeDawkins marked this pull request as ready for review March 29, 2021 16:22
@JakeDawkins JakeDawkins changed the title WIP release docs Release docs Mar 29, 2021
@JakeDawkins JakeDawkins added the docs 📝 improvements or additions to docs on the docs site label Mar 29, 2021
@JakeDawkins JakeDawkins added this to the March 30 - GraphQL Summit milestone Mar 29, 2021
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper left a comment

Choose a reason for hiding this comment

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

looking great!!

[data core]: https://apollo-specs.github.io/#def-data-core
[referencing specifications]: https://apollo-specs.github.io/core/draft/pre-0#core__Using

Currently, supergraph schemas are in preview, and can't be directly consumed by anything. In the future, a supergraph schema document will be consumable by [apollo gateway](https://www.apollographql.com/docs/federation/gateway/), providing all the information needed to execute operations against a supergraph.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Currently, supergraph schemas are in preview, and can't be directly consumed by anything. In the future, a supergraph schema document will be consumable by [apollo gateway](https://www.apollographql.com/docs/federation/gateway/), providing all the information needed to execute operations against a supergraph.
Currently, supergraph schemas are in preview, and can't be directly consumed by anything. In the future, a supergraph schema document will be consumable by [Apollo gateway](https://www.apollographql.com/docs/federation/gateway/), providing all the information needed to execute operations against a supergraph.

Rover's `supergraph compose` command provides all the tools you need to compose a supergraph's schema locally.

```bash
rover supergraph compose --config ./subgraphs.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
rover supergraph compose --config ./subgraphs.yaml
rover supergraph compose --config ./supergraph.yaml


The config format that `supergraph compose` expects is a slim YAML file, containing a list of subgraphs, their public facing urls to route requests to, and where Rover can find their schemas.

Currently, **only federated SDL files are accepted for schema sources**. If you don't have a schema file for a subgraph, you can pipe the output of [`subgraph introspect`](./subgraphs#fetching-via-enhanced-introspection) to a file (see [using stdout](./essentials#using-stdout) for more).
Copy link
Contributor

Choose a reason for hiding this comment

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

You could also reasonably pipe rover subgraph fetch to a file if a subgraph already exists in the registry

films:
routing_url: https://films.example.com
schema:
file: ./good-films.graphql
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
file: ./good-films.graphql
file: ./films.graphql

people:
routing_url: https://people.example.com
schema:
file: ./good-people.graphql
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
file: ./good-people.graphql
file: ./people.graphql

@@ -8,21 +8,28 @@ module.exports = {
options: {
...themeOptions,
root: __dirname,
subtitle: 'Rover CLI',
subtitle: 'Rover CLI (Preview)',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should we drop the CLI in the name or do you think that adds clarity @StephenBarlow?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should keep "CLI" for the time being, until Rover has better name recognition via full release. Will be especially helpful in the docset switcher

the result of a command, whether that be SDL output, checks errors, or
composition errors.
```mermaid
graph BT;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

❤️

gateway --- serviceA & serviceB & serviceC;
```

Rover commands that interact with subgraphs begin with `rover supergraph`. These commands primarily deal with composition of a **supergraph schema** that adheres to the [supergraph schema specification](https://apollo-specs.github.io/core/draft/pre-0).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
Rover commands that interact with subgraphs begin with `rover supergraph`. These commands primarily deal with composition of a **supergraph schema** that adheres to the [supergraph schema specification](https://apollo-specs.github.io/core/draft/pre-0).
Rover commands that interact with supergraphs begin with `rover supergraph`. These commands primarily deal with composition of a **supergraph schema** that adheres to the [supergraph schema specification](https://apollo-specs.github.io/core/draft/pre-0).

@EverlastingBugstopper EverlastingBugstopper merged commit 8e65d8c into main Mar 30, 2021
@EverlastingBugstopper EverlastingBugstopper deleted the jake/release-docs branch March 30, 2021 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs 📝 improvements or additions to docs on the docs site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants