Skip to content

Document remaining public declarations across the smaller libraries - #11542

Open
timotheeguerin wants to merge 1 commit into
mainfrom
docs/library-docs-small-libs
Open

Document remaining public declarations across the smaller libraries#11542
timotheeguerin wants to merge 1 commit into
mainfrom
docs/library-docs-small-libs

Conversation

@timotheeguerin

@timotheeguerin timotheeguerin commented Aug 4, 2026

Copy link
Copy Markdown
Member

Clears the last documentation gaps in protobuf, graphql, events and http-client, which brings every library in the repo to zero violations.

The protobuf part fixes a visible bug on the published docs. Markdown link anchors like [marked reserved](#@TypeSpec.Protobuf.reserve) have their @ read as a doc tag, so the reserve page today renders a bullet that stops mid-link:

- not fall within any range that was [marked reserved](#

Percent-encoding the @ keeps the anchor working and restores the sentence.

Finally, graphql and protobuf ship .tsp but never ran lint-typespec-library; they are now wired into their builds like every other library.

Prerequisite for #1229 and #2090. The library linter rule that catches this is in #11543, which must merge after this one: lint-typespec-library runs with --warn-as-error inside every package's build, so the docs have to exist before the rule is turned on.

Independent of the other documentation PRs (#11539, #11540, #11541) — they touch disjoint packages and can merge in any order.

@azure-sdk-automation

azure-sdk-automation Bot commented Aug 4, 2026

Copy link
Copy Markdown

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

@timotheeguerin
timotheeguerin marked this pull request as ready for review August 4, 2026 21:22
@swatkatz

swatkatz commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

LGTM for GraphQL

@pkg-pr-new

pkg-pr-new Bot commented Aug 5, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/microsoft/typespec/@typespec/events@11542
npm i https://pkg.pr.new/microsoft/typespec/@typespec/graphql@11542
npm i https://pkg.pr.new/microsoft/typespec/@typespec/http-client@11542
npm i https://pkg.pr.new/microsoft/typespec/@typespec/protobuf@11542

commit: f084f82

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

All changed packages have been documented.

  • @typespec/events
  • @typespec/graphql
  • @typespec/http-client
  • @typespec/protobuf
Show changes

@typespec/events - internal ✏️

Add missing documentation on public declarations and members

@typespec/graphql - internal ✏️

Add missing documentation on public declarations and members

@typespec/http-client - internal ✏️

Add missing documentation on public declarations and members

@typespec/protobuf - internal ✏️

Add missing documentation on public declarations and members

…declarations

Documents the last undocumented declarations, members and decorator
parameters across the smaller libraries.

In protobuf, escapes the `@` in the markdown link anchors: the doc parser
read `#@TypeSpec.Protobuf.reserve` as a tag, which truncated the published
docs mid-link to `[marked reserved](#`.

Also wires `lint-typespec-library` into the build of `graphql` and
`protobuf`, which ship .tsp but were not linted.

Prerequisite for #1229 and #2090.
@timotheeguerin
timotheeguerin force-pushed the docs/library-docs-small-libs branch from 0212b44 to f084f82 Compare August 5, 2026 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:graphql Issues for @typespec/graphql emitter emitter:protobuf The protobuf emitter meta:website TypeSpec.io updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants