chain skillmap npm install

[pelikhan]

The requirement to run npm install in /skillmaps is undocumented and very annoying. This should make it automatic.

[riknoll]

@pelikhan is gulp not doing this for you? are you on an old branch maybe?

I added this to the gulpfile recently:

https://github.com/microsoft/pxt/commit/b2b87663e6514fecbcfad49cf435e3b43a4aec95

[pelikhan]

I guess I had an old distro, and running "./pxt> npm install" did not trigger running "./pxt/skillmaps> npm install" as well

[thsparks]

@pelikhan is gulp not doing this for you? are you on an old branch maybe?

I added this to the gulpfile recently:

b2b8766

After the React 17 update, I did actually have to manually run npm install inside the skillmap folder for "gulp watch" to work. Haven't looked into why yet (it's on my to do list), but thought I should mention it. Seems like this change could be helpful, though.

[pelikhan]

I did 'npm install'. what gulp command should i run?

[eanders-ms]

This has been fixed. Close?