Formatting

[xolotlatoani]

• makes sure to use const and let on as much as possible

• Fixes JSDoc

• Removes YUI Compressor (incompatible with es6)

• Uses Terser to compress the js file (to support es6)

• Updates README command to clean, build and compress the files

• Removes the grunt task "grunt-closure-compiler" as it doesn't seem to be used (it was using YUI instead)

• Removes the JSDoc documentation (compiled docs) as it seems that was never there.

@harrymoore @uzquiano @cloudcms

Sorry I needed to recreate this after running the tests

[uzquiano]

Hi Juan,

I'd like to get this merged in the near term. You put a lot of work in here and it looks really good. We hadn't updated the JS driver's build dependencies and I appreciate what you've done to improve that.

My sense is that we could merge this PR and then further address the NPM audit you mentioned in the other ticket.

Have you been using this version in your code? Or building from master? I'm just curious if the build product here passed tests or if you've tried it out and it seemed to work relatively well. We'll have to run it through the full test suite but based on what I see, I don't see why it wouldn't pass in short order.

Cheers,

Michael

[xolotlatoani]

Hi,

I couldn't really run all the tests.

I followed the instructions but it has errors trying to connect to test.cloudcms.com and starting a server, so I changed the server configuration to localhost and the it has issues with the proxy server.

I've currently been using the npm version, since I could not get the tests to run. Is the documentation to run the tests outdated?

[xolotlatoani]

• Pulled the latest code from Master.

• Changed the proxy server for the tests from test.cloudcms.com to test.cloudcms.net since the .com domain is not responding.

• Still having issues connecting to the proxy server.

[xolotlatoani]

I was able to use Chrome's Local Override feature to load the changes onto CloudCMS and everything seems to be loading just fine.