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

feat(tab): Convert JS to TypeScript #4393

Merged
merged 16 commits into from
Feb 12, 2019
Merged

Conversation

acdvorak
Copy link
Contributor

@acdvorak acdvorak commented Feb 9, 2019

Refs #4225

@kfranqueiro kfranqueiro mentioned this pull request Feb 9, 2019
45 tasks
@codecov-io
Copy link

codecov-io commented Feb 9, 2019

Codecov Report

Merging #4393 into feat/typescript will increase coverage by 0.01%.
The diff coverage is 96.47%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##           feat/typescript    #4393      +/-   ##
===================================================
+ Coverage            98.65%   98.67%   +0.01%     
===================================================
  Files                   94       94              
  Lines                 5962     5972      +10     
  Branches               803      804       +1     
===================================================
+ Hits                  5882     5893      +11     
+ Misses                  79       78       -1     
  Partials                 1        1
Impacted Files Coverage Δ
packages/mdc-tab/foundation.ts 100% <100%> (ø)
packages/mdc-tab/index.ts 95.08% <95.08%> (ø)

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 5f50d60...8181828. Read the comment docs.

@moog16
Copy link
Contributor

moog16 commented Feb 11, 2019

are there supposed to be other commits? Not much for the tab package in this PR.

@acdvorak
Copy link
Contributor Author

Woops, looks like I missed some Closure syntax in the README. Fixed.

@moog16
Copy link
Contributor

moog16 commented Feb 11, 2019

oh thanks for catching that I haven't been checking the readmes

packages/mdc-tab/index.ts Outdated Show resolved Hide resolved
packages/mdc-tab/index.ts Outdated Show resolved Hide resolved
packages/mdc-tab/types.ts Outdated Show resolved Hide resolved
@mdc-web-bot
Copy link
Collaborator

All 624 screenshot tests passed for commit 8fd2a0e vs. feat/typescript! 💯🎉

@acdvorak acdvorak merged commit ab9879d into feat/typescript Feb 12, 2019
@mdc-web-bot
Copy link
Collaborator

All 624 screenshot tests passed for commit 53da022 vs. feat/typescript! 💯🎉

@acdvorak acdvorak deleted the feat/typescript--tab branch February 12, 2019 05:10
@mdc-web-bot
Copy link
Collaborator

All 624 screenshot tests passed for commit 8181828 vs. feat/typescript! 💯🎉

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