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

Rax: Migrate to TS #13450

Merged
merged 2 commits into from
Jan 13, 2021
Merged

Rax: Migrate to TS #13450

merged 2 commits into from
Jan 13, 2021

Conversation

gaetanmaisse
Copy link
Member

@gaetanmaisse gaetanmaisse commented Dec 14, 2020

Issue: This is the last stone of #5030 😱 🥳 🎉
(I would love to merge this PR and close #5030 myself as it was the issue of my first contribution to SB 😍 )

What I did

Fastly migrate @storybook/rax to TS as I want to close the big TS Migration Issue before moving to improve types of @storybook/core.

How to test

  • CI should be 🟢 + added the extended e2e test suite as there is one E2E for Rax 🥳

@gaetanmaisse gaetanmaisse added maintenance User-facing maintenance tasks app: rax run e2e extended test suite Run the e2e extended test suite in CircleCI workflows labels Dec 14, 2020
@gaetanmaisse gaetanmaisse requested a review from a team December 14, 2020 21:01
@gaetanmaisse
Copy link
Member Author

gaetanmaisse commented Dec 14, 2020

@gaetanmaisse gaetanmaisse force-pushed the tech/migrate-rax-to-ts branch 2 times, most recently from 54b9ed7 to 82d7de6 Compare December 18, 2020 14:18
@shilman shilman changed the title Rax: migrate to TS Rax: Migrate to TS Dec 20, 2020
@gaetanmaisse gaetanmaisse force-pushed the tech/migrate-rax-to-ts branch from 82d7de6 to a13af7b Compare January 6, 2021 20:29
Copy link
Contributor

@ThibaudAV ThibaudAV left a comment

Choose a reason for hiding this comment

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

LGTM

wow 😱 🥳 🎉

I can imagine your face like this right now ^^
image

@gaetanmaisse gaetanmaisse force-pushed the tech/migrate-rax-to-ts branch from a13af7b to e998038 Compare January 13, 2021 19:07
@gaetanmaisse gaetanmaisse merged commit ef55773 into next Jan 13, 2021
@gaetanmaisse gaetanmaisse deleted the tech/migrate-rax-to-ts branch January 13, 2021 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks run e2e extended test suite Run the e2e extended test suite in CircleCI workflows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeScript migration organization
3 participants