-
Notifications
You must be signed in to change notification settings - Fork 987
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
Fixes #21578 - Cmp.PropTypes to Cmp.propTypes #4978
Fixes #21578 - Cmp.PropTypes to Cmp.propTypes #4978
Conversation
Any idea why Travis is failing because of mismatched snapshots? The changes shouldn't affect the snapshots and it is not failing locally. |
@sharvit this is the same failure @danseethaler is facing on #4944 - did you figure out which npm package shoudld be pinned? |
@ohadlevy - check out my comment on @danseethaler pr #4944 |
Based on those pull requests: theforeman#4944 theforeman#4978 Upgrading the enzyme packages and update the snapshots.
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.
This looks fine to me, approved pending passing tests.
@sharvit please rebase |
Chart, ChartBox and StatisticsChartsList are using prop-types in a wrong way. Chart is declaring about prop-types it is never used. http://projects.theforeman.org/issues/21578
8ac48fa
to
3412d4e
Compare
Rebased and passed all tests. |
Based on those pull requests: theforeman#4944 theforeman#4978 Upgrading the enzyme packages and update the snapshots.
Chart, ChartBox and StatisticsChartsList are using prop-types in a wrong
way.
Chart is declaring about prop-types it is never used.
http://projects.theforeman.org/issues/21578