Include JSONFormatter in bundle

[mhamann]

• Converted build to use webpack
• Upgraded babel version
• Included JSONFormatter in build by default
• Removed bower deps in favor of npm

[mohsen1]

very nice! I'll merge it tonight

[mhamann]

@mohsen1 and updates on merging this and publishing a new version to npm?

[mohsen1]

Hey @mhamann this is great but we need to make sure it works end-to-end
I couldn't make a build and confirm this is working and don't have enough time to dig into it.

I would like to move away from bower and gulp and instead build it with webpack and test it with Jest

Can you look into it?

Thanks!

[mhamann]

@mohsen1 yeah, i can try to find some time to work on that. (cc: @chrisdudley - maybe someone from your team could collab w/ me on this)

I see the PR was reverted. Did something break?

I did run some manual tests to ensure this worked and solved the issue--and it did. But certainly could break people if they were depending on the bower side of things.

This probably deserves a 2.0.0 version bump.

[mohsen1]

The bower dependencies were still in the demo app. What I wish I could do is to have webpack dev server for demo and webpack for building the js/css assets

[mhamann]

Gotcha. Webpack dev server shouldn't be a problem.

[mitar]

I tried the version at this commit (1f87948) and I can use it in my code with npm + browserify with:

import JSONSchemaView from 'json-schema-view-js';

On the current master branch (with commit reverted) I cannot because it does not find ./helpers.js during browserify bundling.

One thing I had to add was json-formatter-js dependency, which was not automatically installed.

[mitar]

This dependencies key overrides the dependencies key above, which contains json-formatter-js. You have a duplicate JSON key here.