-
Notifications
You must be signed in to change notification settings - Fork 111
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
Jest 19 #422
Jest 19 #422
Conversation
When testing the starter-kyts, this message from Jest shows up (for both starter-kyts):
Not entirely sure why 🤔 The tests pass though |
packages/kyt-core/package.json
Outdated
"react": "^15.4.2", | ||
"react-addons-test-utils": "^15.4.2", | ||
"react-dom": "^15.4.2" | ||
} |
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.
Had to add these because of enzyme-to-json serializer. It's a devDependency so it shouldn't affect anything in the published packages.
* master: Upgrade stylelint (#417) Update to Webpack 2.2 (#396) Version and install starter-kyts with npm (#425) adds prep for 0.4.1 (#428) fixes static public path for noServer configuration (#427) Adds local-path argument for testing local starter-kyt setup (#403) # Conflicts: # packages/kyt-core/package.json # packages/kyt-core/yarn.lock # packages/kyt-starter-static/starter-src/yarn.lock # packages/kyt-starter-universal/starter-src/src/yarn.lock # yarn.lock
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.
Hey @tizmagik this looks good. Can you add a note about the changes in CHANGELOG.md ?
Also are there any breaking changes users will need to know about? If so we should start a docs branch for 0.5.0 and make updates.
Closes #394