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

Add docc docs for grpc core #2016

Merged
merged 1 commit into from
Aug 14, 2024
Merged

Add docc docs for grpc core #2016

merged 1 commit into from
Aug 14, 2024

Conversation

glbrntt
Copy link
Collaborator

@glbrntt glbrntt commented Aug 13, 2024

Motivation:

As we move towards v2 we need to make a clearer separation between v1 and v2 docs to avoid confusion.

Modifications:

  • Remove v1 specific content out of README.md (it is duplicated in the v1 DocC docs) and provide a clearer update on the state of the project. This also points users to the v1 docs on Swift Package Index where the duplicated v1 content lives.
  • Add a doc catalog to gRPC core with a very high level description of the package products
  • Add a "Benchmarks" tutorial to the v2 doc catalog (it was content removed from the README)

Result:

  • DocC catalog in place for v2
  • README is clearer

Motivation:

As we move towards v2 we need to make a clearer separation between v1
and v2 docs to avoid confusion.

Modifications:

- Remove v1 specific content out of README.md (it is duplicated in the
  v1 DocC docs) and provide a clearer update on the state of the
  project. This also points users to the v1 docs on Swift Package Index
  where the duplicated v1 content lives.
- Add a doc catalog to gRPC core with a very high level description of
  the package products
- Add a "Benchmarks"  tutorial to the v2 doc catalog (it was content
  removed from the README)

Result:

- DocC catalog in place for v2
- README is clearer
@glbrntt glbrntt added the semver/none No version bump required. label Aug 13, 2024
@glbrntt glbrntt requested a review from gjcairo August 13, 2024 15:04
transports built on top of SwiftNIO's Network.framework abstraction, `NIOTransportServices`.
- `GRPCProtobuf` provides serialization and deserialization components for `SwiftProtobuf`.

## Topics
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this section supposed to be empty or do we want to remove it / add something to it?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It's not empty, it has the "Getting involved" subsection. There will be more subsections (e.g. tutorials/articles) in the future too.

## Topics is also magic: if you don't add it then DocC will add it and list all articles etc.
underneath it (as well as API docs).

If you add it manually it'll add in everything that you didn't explicitly add, so at the moment it'll look like this:

Screenshot 2024-08-14 at 13 22 47

Copy link
Collaborator

@gjcairo gjcairo left a comment

Choose a reason for hiding this comment

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

LGTM, just a small comment

@glbrntt glbrntt merged commit f391779 into grpc:main Aug 14, 2024
15 of 17 checks passed
@glbrntt glbrntt deleted the v2/docc branch August 14, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants