Type check docs examples #3710
Type check docs examples #3710
Conversation
|
Build successful! 🎉 |
This comment was marked as outdated.
This comment was marked as outdated.
| @@ -277,53 +277,8 @@ function DropIndicator(props) { | |||
|
|
|||
| Now that the `DropIndicator` component is implemented, we can render an instance between each item in the list. This uses the `before` drop position by default, except for after the last item in the list. | |||
|
|
|||
| ```tsx example render=false | |||
| ///- begin collapse -/// | |||
| function ListBox(props) { | |||
There was a problem hiding this comment.
This was unused. The real implementation is in the below example.
|
|
||
| return ( | ||
| <> | ||
| <p>Current tab id: {tabId}</p> | ||
| <Tabs aria-label="History of Ancient Rome" items={tabs} onSelectionChange={setTabId}> | ||
| <TabList> | ||
| {item => ( | ||
| {(item: Tab) => ( |
There was a problem hiding this comment.
These are annoying. Because we designed the API with items on Tabs rather than TabList and TabPanels, this will always be unknown by default without an annotation.
| function Example() { | ||
| let [selected, setSelected] = React.useState(new Set(['cheese'])); | ||
| let [selected, setSelected] = React.useState<Selection>(new Set(['cheese'])); |
There was a problem hiding this comment.
These are annoying, and caused by onSelectionChange which may emit "all" | Set<Key>, but the type of this variable is only inferred to be Set<string> so it always requires an annotation.
…m into docs-ts # Conflicts: # packages/@adobe/react-spectrum/package.json # packages/@adobe/react-spectrum/src/index.ts # packages/@react-aria/dnd/src/index.ts # packages/@react-aria/overlays/docs/useOverlayTrigger.mdx # packages/@react-spectrum/autocomplete/src/SearchAutocomplete.tsx # packages/@react-spectrum/list/docs/ListView.mdx # packages/dev/parcel-transformer-mdx-docs/MDXTransformer.js
|
Build successful! 🎉 |
# Conflicts: # packages/@adobe/react-spectrum/package.json # packages/@react-spectrum/actionbar/docs/ActionBar.mdx
|
Build successful! 🎉 |
|
Build successful! 🎉 |
|
Build successful! 🎉 |
|
## API Changes
unknown top level export { type: 'identifier', name: 'Column' } |
Part of #865
This adds a script that extracts all of the examples from the docs MDX files into TSX files, and then runs TypeScript over them. This will run in CI. It found a number of issues, such as unused variables, missing props, etc. So far, I have only fixed the existing type errors, adding annotations where needed. Eventually, we can add type annotations to all examples.
We may want to eventually have a toggle to switch between TS and JS. For now, only TS is displayed.