update gulpfile

[vmanasiev]

Signed-off-by: Valentin [email protected]

Description

The build fails on windows machines for two reasons

• The child process does not always return return code ( returns null), adding parameter shell:true fixes this issue
• The pbjs and pbts scripts found in the protobufjs/bin folder are only shell compatible. There are two ways to fix the issue. Either execute those with node and pass pbjs path as parameter or change the path to nodejs_modules/.bin/pbjs where you can also find pbjs.cmd and pbts.cmd (just as it is done with tsc and vsce

[welcome]

Thank you for taking the time to contribute! We'll check this out soon and let you know if we have any questions or feedback.
If you haven't already, please be sure to check out the contributing guidelines.

[codecov]

Codecov Report

Merging #67 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #67   +/-   ##
=======================================
  Coverage   52.09%   52.09%           
=======================================
  Files          39       39           
  Lines         741      741           
  Branches      101      101           
=======================================
  Hits          386      386           
  Misses        355      355