-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Conversation
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.
Nice fix! Just a little eager on the version number for @material/animation
. Update that and it should be good to go!
packages/mdc-ripple/package.json
Outdated
@@ -15,6 +15,7 @@ | |||
}, | |||
"dependencies": { | |||
"@material/base": "^0.35.0", | |||
"@material/theme": "^0.35.0" | |||
"@material/theme": "^0.35.0", | |||
"@material/animation": "^0.35.0" |
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 should be 0.34.0
. We haven't released 0.35.0
of animation yet.
"version": "0.34.0", |
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.
That's why the CI tests were failing, too.
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.
Done. Thanks!
…13' into fix_mdc_list_missing_dep_issue2713
Codecov Report
@@ Coverage Diff @@
## master #2795 +/- ##
=======================================
Coverage 98.48% 98.48%
=======================================
Files 98 98
Lines 4232 4232
Branches 538 538
=======================================
Hits 4168 4168
Misses 64 64 Continue to review full report at Codecov.
|
…13' into fix_mdc_list_missing_dep_issue2713
Fixes #2713