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

Add TypeScript CRA example #5120

Merged
merged 15 commits into from Jan 9, 2019
Merged

Add TypeScript CRA example #5120

merged 15 commits into from Jan 9, 2019

Conversation

mucsi96
Copy link

@mucsi96 mucsi96 commented Dec 29, 2018

Issue:

What I did

I have created an example incorporating CRA2, TypeScript, TSLint, Jest, Enzyme

image

Closes #5006
Closes #5011

@igor-dv @ndelangen Please review

@mucsi96 mucsi96 changed the title Ts cra example Add TypeScript CRA example Dec 29, 2018
@codecov
Copy link

codecov bot commented Dec 29, 2018

Codecov Report

Merging #5120 into next will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             next    #5120   +/-   ##
=======================================
  Coverage   35.23%   35.23%           
=======================================
  Files         596      596           
  Lines        7401     7401           
  Branches     1010     1010           
=======================================
  Hits         2608     2608           
  Misses       4285     4285           
  Partials      508      508

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3698c88...47b31a7. Read the comment docs.

@igor-dv igor-dv added maintenance User-facing maintenance tasks typescript cra Prioritize create-react-app compatibility labels Dec 31, 2018
@igor-dv
Copy link
Member

igor-dv commented Dec 31, 2018

We need to setup netlify and CI, other than that LGTM.

Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

This is great. However, it looks like the acceptance test is not set up properly since it's not rendering right in Chromatic. Not sure what's going on because the code looks good to me.

Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

see comment above

@Hypnosphi
Copy link
Member

@shilman
Copy link
Member

shilman commented Jan 6, 2019

Hmm Chromatic is still not rendering it properly. Does it work on your dev machine?

@Hypnosphi
Copy link
Member

Hypnosphi commented Jan 7, 2019

The problem is that right now teamcity doesn't build the new example. You need to add an entry here.
https://github.com/storybooks/storybook/blob/next/.teamcity/OpenSourceProjects_Storybook/buildTypes/StorybookApp.kt#L10
And unfortunatelty you need to do it directly on next branch first because that's the branch that forms the builds graph. You need to set merged flag to false when doing that in next, and then to true in this branch.

@mucsi96
Copy link
Author

mucsi96 commented Jan 7, 2019

Can you help with that? I not sure I understand as this is my first contribution to storybook. Probably I don’t have rights for that.

@shilman
Copy link
Member

shilman commented Jan 8, 2019

Thanks @Hypnosphi for taking care of the teamcity -- looks great!

Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

@mucsi96 Thanks for your patience on this and your contribution. It looks like the yarn lockfile is dirty. Can you update that and I'll merge?

@mucsi96
Copy link
Author

mucsi96 commented Jan 8, 2019

Thx @shilman ! Thx @Hypnosphi for helping out with TeamCity! I updated the lockfile. Unfortunately the CRA TS build is failing with

Failed to resolve artifact dependency <Open-source projects / Storybook / Bootstrap, build #2331 [id 1871311]>: No files matched for patterns "dist.zip" from <Open-source projects / Storybook / Bootstrap, build #2331

Can you help with resolving this?

@shilman
Copy link
Member

shilman commented Jan 9, 2019

Looks like it's building now, so I'm merging. Thanks for hanging in there!

@shilman shilman merged commit 2436c9d into storybookjs:next Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cra Prioritize create-react-app compatibility maintenance User-facing maintenance tasks typescript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CRA2 TypeScript example
4 participants