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

fix(drawer): Remove unnecessary Closure annotation #3935

Merged
merged 2 commits into from
Oct 19, 2018

Conversation

patrickrodee
Copy link
Contributor

Modal Drawer extends Dismissible Drawer. There's no need for an adapter generic assignment.

Modal Drawer extends Dismissible Drawer. There's no need for an adapter generic assignment.
@mdc-web-bot
Copy link
Collaborator

All 560 screenshot tests passed for commit a1cfb9c vs. master! 💯🎉

@codecov-io
Copy link

codecov-io commented Oct 19, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@bf98ec7). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3935   +/-   ##
=========================================
  Coverage          ?   98.48%           
=========================================
  Files             ?      120           
  Lines             ?     5227           
  Branches          ?      657           
=========================================
  Hits              ?     5148           
  Misses            ?       79           
  Partials          ?        0
Impacted Files Coverage Δ
packages/mdc-drawer/modal/foundation.js 100% <ø> (ø)

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 bf98ec7...93ca625. Read the comment docs.

@mdc-web-bot
Copy link
Collaborator

All 564 screenshot tests passed for commit 93ca625 vs. master! 💯🎉

@williamernest williamernest merged commit 61128be into master Oct 19, 2018
@williamernest williamernest deleted the patrickrodee-patch-1 branch October 19, 2018 19:15
@jamesmfriedman jamesmfriedman mentioned this pull request Oct 30, 2018
27 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants