Replace gulp-webserver with gulp-connect. Fixes issue #195.

[djasnowski]

This adheres issue #195 and successfully replaces said NPM package (which had 2 bad vulns). When I ran npm audit, it had no more security concerns.

➜  ~/code/oss/morgan-stanley/desktopJS git:(master) npm audit

                       === npm audit security report ===

found 0 vulnerabilities
 in 50045 scanned packages

---

[djasnowski]

@bingenito I hope this can fix it. A new PR with the commit (merged in one) and the covered by the DCO statement.

[codecov]

Codecov Report

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

@@           Coverage Diff           @@
##           master     #221   +/-   ##
=======================================
  Coverage   93.44%   93.44%           
=======================================
  Files          16       16           
  Lines        1479     1479           
  Branches      249      249           
=======================================
  Hits         1382     1382           
  Misses         97       97

[bingenito]

lgtm

[bingenito]

@naknode Give me a day or so to work out future PR merge strategy on the repo now that we are dco. Whether it should be merge vs squash, etc. I had been doing squash and merge in the past.

[djasnowski]

Same here, it was always simple but DCO is better. Thanks for your speedy input.

[bingenito]

Fixes #195