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

build: package bundler hint to excluded node modules #132

Merged
merged 1 commit into from
Oct 17, 2018

Conversation

zachferland
Copy link
Contributor

XMLHttpRequest already does this, and it should be a sufficient hint for any bundler to not include the node modules, but it is not working for a number of people. This makes it more explicit, by specifying the same in our own package.

@codecov
Copy link

codecov bot commented Oct 17, 2018

Codecov Report

Merging #132 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #132   +/-   ##
========================================
  Coverage    84.61%   84.61%           
========================================
  Files            6        6           
  Lines          377      377           
  Branches        51       51           
========================================
  Hits           319      319           
  Misses          50       50           
  Partials         8        8

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 d6b2a95...7d20f55. Read the comment docs.

Copy link
Contributor

@oed oed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@zachferland zachferland merged commit 2f5313e into develop Oct 17, 2018
@zachferland zachferland mentioned this pull request Oct 17, 2018
@c-castillo c-castillo deleted the build/node-modules-browser branch October 25, 2018 17:49
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.

2 participants