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

fix(core): Fix canary release by excluding test files from default tsconfig #5317

Merged
merged 1 commit into from
Dec 9, 2019

Conversation

joyzhong
Copy link
Contributor

@joyzhong joyzhong commented Dec 9, 2019

For some reason, test files in the tsconfig are causing package/<component>/index.js not to be generated when running npm run build:esmodules, leading to a failure in the canary release: https://github.com/material-components/material-components-web/runs/340549500

@codecov-io
Copy link

codecov-io commented Dec 9, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #5317   +/-   ##
======================================
  Coverage    98.3%   98.3%           
======================================
  Files         162     162           
  Lines        6311    6311           
  Branches      866     866           
======================================
  Hits         6204    6204           
  Misses        106     106           
  Partials        1       1

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 a0da497...282a257. Read the comment docs.

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.

LGTM, title needs an update but otherwise good to go!

@joyzhong joyzhong changed the title Exclude test files from default tsconfig. Fix canary release by excluding test files from default tsconfig. Dec 9, 2019
@joyzhong joyzhong changed the title Fix canary release by excluding test files from default tsconfig. fix(core): Fix canary release by excluding test files from default tsconfig. Dec 9, 2019
@mdc-web-bot
Copy link
Collaborator

All 731 screenshot tests passed for commit 282a257 vs. master! 💯🎉

@joyzhong joyzhong changed the title fix(core): Fix canary release by excluding test files from default tsconfig. fix(core): Fix canary release by excluding test files from default tsconfig Dec 9, 2019
@joyzhong joyzhong merged commit c916008 into master Dec 9, 2019
@joyzhong joyzhong deleted the refactor/checkbox_test branch December 9, 2019 22:42
This was referenced Feb 13, 2020
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.

5 participants