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

Tech: TypeScript should check stories and tests as well #19831

Merged
merged 5 commits into from
Nov 14, 2022

Conversation

ndelangen
Copy link
Member

@ndelangen ndelangen commented Nov 14, 2022

What I did

  • I modified storyshots to mock the renderer, and modify it so has the old API
  • I enabled typescript checking for testfiles and stories
  • I fixed the issues resulted from that

@ndelangen ndelangen self-assigned this Nov 14, 2022
@ndelangen ndelangen added the maintenance User-facing maintenance tasks label Nov 14, 2022
@ndelangen ndelangen changed the title Tech: typesscript should typescript stories and tests as well Tech: TypeScript should typescript stories and tests as well Nov 14, 2022
@ndelangen ndelangen changed the title Tech: TypeScript should typescript stories and tests as well Tech: TypeScript should check stories and tests as well Nov 14, 2022
@ndelangen
Copy link
Member Author

@shilman self merging

@ndelangen ndelangen merged commit 1e9e0c4 into next Nov 14, 2022
@ndelangen ndelangen deleted the tech/legacy-api-storyshots branch November 14, 2022 16:35
@IanVS
Copy link
Member

IanVS commented Nov 15, 2022

I enabled typescript checking for testfiles and stories

Nice! I noticed a day or two ago that we weren't typechecking test files, and wanted to dig into why. Then along you come and fix it, @ndelangen! 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants