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
Migrate @storybook/angular to typescript #6570
Conversation
This pull request is automatically deployed with Now. Latest deployment for this branch: https://monorepo-git-ts-migration-app-angular.storybook.now.sh |
tsc wants all imports to be under rootDir
@@ -1,5 +1,5 @@ | |||
import { getAngularCliWebpackConfigOptions } from './angular-cli_config'; | |||
import angularJson from '../../../../examples/angular-cli/angular.json'; | |||
import { getAngularCliWebpackConfigOptions } from '../angular-cli_config'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just checking what is happening around, do we have some new convention to put tests in the __tests
dir ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like colocation of tests, until the folder gets like 6+ files, then having a tests
folder is kinda nice. I don't care enough about it either way really. 👍
e28b3bd
to
6a37e95
Compare
6a37e95
to
e079c89
Compare
# Conflicts: # app/angular/src/client/preview/index.js # scripts/compile-babel.js # yarn.lock
e079c89
to
997ce27
Compare
195ff88
to
c87d5ae
Compare
Issue: #5030