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

fix(menu): Rename test files #2168

Merged
merged 1 commit into from
Feb 13, 2018
Merged

Conversation

williamernest
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Feb 2, 2018

Codecov Report

Merging #2168 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2168      +/-   ##
==========================================
+ Coverage   99.13%   99.14%   +<.01%     
==========================================
  Files          90       90              
  Lines        3830     3840      +10     
  Branches      496      495       -1     
==========================================
+ Hits         3797     3807      +10     
  Misses         33       33
Impacted Files Coverage Δ
packages/mdc-select/foundation.js 100% <0%> (ø) ⬆️
packages/mdc-checkbox/index.js 100% <0%> (ø) ⬆️
packages/mdc-ripple/foundation.js 100% <0%> (ø) ⬆️
packages/mdc-icon-toggle/index.js 100% <0%> (ø) ⬆️
packages/mdc-menu/constants.js 100% <0%> (ø) ⬆️
packages/mdc-ripple/index.js 100% <0%> (ø) ⬆️
packages/mdc-menu/foundation.js 99.35% <0%> (ø) ⬆️
packages/mdc-menu/index.js 93.97% <0%> (+0.07%) ⬆️
packages/mdc-radio/index.js 97.67% <0%> (+0.23%) ⬆️

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 665cf12...02ca397. Read the comment docs.

@kfranqueiro
Copy link
Contributor

This is presumably conflicting now due to merging #1756. Not sure if it's simpler to just re-do the rename from master now.

@williamernest williamernest force-pushed the fix/menu/rename-test-files branch from f2c89fe to 02ca397 Compare February 13, 2018 17:21
@williamernest williamernest merged commit 5ea5c2f into master Feb 13, 2018
@williamernest williamernest deleted the fix/menu/rename-test-files branch February 13, 2018 18:08
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