Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

fix(ripple): Fix missing dependency #2795

Merged
merged 11 commits into from
May 25, 2018

Conversation

abhiomkar
Copy link
Collaborator

Fixes #2713

@abhiomkar abhiomkar requested a review from kfranqueiro May 23, 2018 20:03
@patrickrodee patrickrodee self-assigned this May 24, 2018
Copy link
Contributor

@patrickrodee patrickrodee left a 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!

@@ -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"
Copy link
Contributor

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.

Copy link
Contributor

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.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done. Thanks!

@codecov-io
Copy link

codecov-io commented May 24, 2018

Codecov Report

Merging #2795 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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.

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

@abhiomkar abhiomkar merged commit 16a6890 into master May 25, 2018
@kfranqueiro kfranqueiro deleted the fix_mdc_list_missing_dep_issue2713 branch August 1, 2018 16:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants