Skip to content

Conversation

@tjprescott
Copy link
Member

@tjprescott tjprescott commented Apr 24, 2023

This PR make it so that docs generation fails if anything that should have docs does not. As a result, it will also have a ton of doc fixes.

This also adds ref doc generation for scalars and fixes a logic bug in the generation of docs.

TODO

  • decorators
  • decorator parameters
  • template parameters
  • models
  • unions
  • enums
  • interfaces
  • operations
  • scalars

Example output:

Issues in @typespec/compiler:
Missing documentation for template parameter 'Array.T'.
Missing documentation for template parameter 'Record.T'.
Missing documentation for template parameter 'OptionalProperties.T'.
Missing documentation for template parameter 'UpdateableProperties.T'.
Missing documentation for template parameter 'OmitProperties.T'.
Missing documentation for template parameter 'OmitProperties.TKeys'.
Missing documentation for template parameter 'OmitDefaults.T'.
Missing documentation for template parameter 'DefaultKeyVisibility.T'.
Missing documentation for template parameter 'DefaultKeyVisibility.Visibility'.
Missing documentation for decorator '@withUpdateableProperties'.
Missing documentation for decorator parameter '@withoutOmittedProperties.omit'.
Missing documentation for decorator '@withoutOmittedProperties'.
Missing documentation for decorator '@withoutDefaultValues'.
Missing documentation for decorator parameter '@withDefaultKeyVisibility.visibility'.
Missing documentation for decorator '@withDefaultKeyVisibility'.
Missing documentation for decorator parameter '@friendlyName.formatArgs'.
Missing documentation for decorator parameter '@key.altName'.
Missing documentation for decorator parameter '@discriminator.propertyName'.
Missing documentation for decorator parameter '@visibility.visibilities'.
Missing documentation for decorator parameter '@withVisibility.visibilities'.
Missing documentation for decorator parameter '@inspectType.text'.
Missing documentation for decorator '@inspectType'.
Missing documentation for decorator parameter '@inspectTypeName.text'.
Missing documentation for decorator '@inspectTypeName'.

Issues in @typespec/http:
Missing documentation for template parameter 'ApiKeyAuth.TLocation'.
Missing documentation for template parameter 'ApiKeyAuth.TName'.
Missing documentation for template parameter 'OAuth2Auth.TFlows'.
Missing documentation for decorator parameter '@includeInapplicableMetadataInPayload.value'.
Missing documentation for decorator parameter '@route.options'.

@github-actions
Copy link
Contributor

Changes in this PR will be published to the following url to try(check status of TypeSpec Pull Request Try It pipeline for publish status):
Playground: https://cadlplayground.z22.web.core.windows.net/prs/1860/

Website: https://cadlwebsite.z1.web.core.windows.net/prs/1860/

@tjprescott tjprescott force-pushed the docs/FixMissingDocs branch 3 times, most recently from fab5c1e to 37c3967 Compare April 25, 2023 19:52
@tjprescott tjprescott force-pushed the docs/FixMissingDocs branch from 11336bb to 06d8370 Compare April 25, 2023 22:11
@tjprescott tjprescott marked this pull request as ready for review April 26, 2023 23:15
@tjprescott tjprescott force-pushed the docs/FixMissingDocs branch 7 times, most recently from 5e380f9 to 47935fb Compare April 27, 2023 19:50
@tjprescott tjprescott force-pushed the docs/FixMissingDocs branch from fd9b62b to ac849f8 Compare April 27, 2023 21:15
@tjprescott tjprescott force-pushed the docs/FixMissingDocs branch from 97d9d6b to 0b74384 Compare April 27, 2023 22:13
@tjprescott tjprescott force-pushed the docs/FixMissingDocs branch from 0b74384 to 7db11e1 Compare April 27, 2023 22:40
@tjprescott tjprescott merged commit 48a1492 into microsoft:main Apr 28, 2023
@tjprescott tjprescott deleted the docs/FixMissingDocs branch April 28, 2023 16:32
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.

2 participants