Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

test: codecov ignore tsx/jsx #482

Merged
merged 2 commits into from
May 13, 2020
Merged

test: codecov ignore tsx/jsx #482

merged 2 commits into from
May 13, 2020

Conversation

ktmud
Copy link
Contributor

@ktmud ktmud commented May 13, 2020

🏆 Enhancements

Follow up on #468 . packages/**/*.js does not exclude .jsx. We need to specifically exclude it.

@ktmud ktmud requested a review from a team as a code owner May 13, 2020 00:45
@vercel
Copy link

vercel bot commented May 13, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/superset/superset-ui/i2ncjjjfg
✅ Preview: https://superset-ui-git-fork-ktmud-codecov.superset.now.sh

@codecov
Copy link

codecov bot commented May 13, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #482   +/-   ##
=======================================
  Coverage   22.74%   22.74%           
=======================================
  Files         276      276           
  Lines        6671     6671           
  Branches      644      644           
=======================================
  Hits         1517     1517           
  Misses       5114     5114           
  Partials       40       40           

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 befe0c7...f08c752. Read the comment docs.

@ktmud ktmud merged commit dacff6c into apache-superset:master May 13, 2020
@ktmud ktmud deleted the codecov branch May 13, 2020 17:58
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.

2 participants