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

[DOC] Add documentation for RFC 0821 items #20272

Merged
merged 1 commit into from
Nov 22, 2022
Merged

Conversation

chriskrycho
Copy link
Contributor

@chriskrycho chriskrycho commented Nov 18, 2022

  • Introduce or relocate documentation for all the @ember/owner types.
  • Make sure all documented owner and routing types correctly express their type hierarchies in the docs so linking and inheritance flags both work correctly.
  • Move the documentation for RouteInfo and Transition to their new modules. Additionally, eliminate the now-empty lib module for the Transition type since it only existed to provide documentation.
  • Update coverage expectation to include the new items.

A note: this, extremely annoyingly, uses @class annotations for a great many types which are not classes. Our current API documentation infrastructure has no idea what types are (reasonably enough: it was built for a pure JS world!), so things like interface or especially type FullName = ... have… hacks… to make them show up correctly in the docs. However, they do work correctly in the docs; I ran against the API Docs app for most of the dev cycle.

- Introduce or relocate documentation for all the `@ember/owner` types.
- Make sure all documented owner and routing types correctly express
  their type hierarchies in the docs so linking and inheritance flags
  both work correctly.
- Move the documentation for `RouteInfo` and `Transition` to their new
  modules. Additionally, eliminate the now-empty `lib` module for the
  `Transition` type since it *only* existed to provide documentation.
- Update coverage expectation to include the new items.
@chriskrycho chriskrycho added Documentation TypeScript Work on Ember’s types labels Nov 18, 2022
@chriskrycho chriskrycho merged commit 74bc861 into master Nov 22, 2022
@chriskrycho chriskrycho deleted the rfc-0821-docs branch November 22, 2022 02:33
Copy link
Contributor

@dfreeman dfreeman left a comment

Choose a reason for hiding this comment

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

Retroactive 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation TypeScript Work on Ember’s types
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants