Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(npm): add ignore list #958

Merged
merged 1 commit into from
Apr 15, 2020
Merged

chore(npm): add ignore list #958

merged 1 commit into from
Apr 15, 2020

Conversation

erisu
Copy link
Member

@erisu erisu commented Apr 15, 2020

Motivation and Context

  • Cleanup the npm package with .npmignore

Description

  • Add .npmignore file

Testing

  • npm t
  • Project
    • cordova create
    • cordova platform add with the cleanup npm package
    • cordova run android
    • cordova build android

Checklist

  • I've run the tests to see all new and existing tests pass

@erisu erisu added this to the 9.0.0 milestone Apr 15, 2020
@erisu erisu requested review from timbru31 and breautek April 15, 2020 02:51
@codecov-io
Copy link

codecov-io commented Apr 15, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #958   +/-   ##
=======================================
  Coverage   65.53%   65.53%           
=======================================
  Files          21       21           
  Lines        1828     1828           
=======================================
  Hits         1198     1198           
  Misses        630      630           

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 71f63d7...a453cbf. Read the comment docs.

@erisu erisu merged commit e4bff28 into apache:master Apr 15, 2020
@erisu erisu deleted the chore/npmignore branch April 15, 2020 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants