fix(umd): add json plugin

[kentcdodds]

What: This adds rollup-plugin-json

Why: Because we broke the rollup build when we started requiring packages that use main as a json file 😄

How: add the plugin, also refactor things a little bit to be cleaner.

cc: @tsriram

[codecov-io]

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #31   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      3    +1     
  Lines          64     62    -2     
  Branches        9      9           
=====================================
- Hits           64     62    -2

Impacted Files	Coverage Δ
src/index.js	100% <ø> (ø)	⬆️
src/dom-elements.js	100% <100%> (ø)

---

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 1293859...68dd350. Read the comment docs.

[kentcdodds]

I added a PR to fix the dependency ci issue: rollup/rollup-plugin-json#25

[kentcdodds]

Merging this as-is considering it's breaking anyone using the UMD build via unpkg.com (like my intro blogpost 😉)

Feel free to still review and we can address any needed changes.

[paulmolluzzo]

I'll review tonight.

Merging this as-is considering it's breaking anyone using the UMD build via unpkg.com (like my intro blogpost 😉)

🙈

[paulmolluzzo]

This looks good to me. I ran npm start validate ✅ and I installed 2.0.1 from npm in a test project ✅ .

Quick fix! 👏 Just that all PRs are going to fail until your PR for the dependency is merged.

[tsriram]

So npm start validate is supposed to fail without this change?

[kentcdodds]

No, npm start validate wont fail based on the UMD build. Though we could make it do that like I do in match-sorter with the dist-test. Not a bad idea actually...