setting runStartupCommands as array

Minor suggestion for improvement: python.dataScience.runStartupCommands must currently be a string. If you use it to run more than one or two commands, the string gets quite long, requiring a lot of horizontal scrolling, making it hard to read and even harder to modify.

How about also allowing that setting to be an array? If an array is detected, simply convert it to string internally:

if (Array.isArray(runStartupCommands)) runStartupCommands = runStartupCommands.join(`\n`)

Then you could have

"python.dataScience.runStartupCommands": [
  "%load_ext autoreload",
  "%autoreload 2",
  "import sys",
  "sys.path.append('${workspaceFolder}')",
  "sys.path.append('${workspaceFolder}/src')",
],

instead of

"python.dataScience.runStartupCommands": "%load_ext autoreload\n%autoreload 2\nimport sys\nsys.path.append('${workspaceFolder}')\nsys.path.append('${workspaceFolder}/src')",

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer, @joyceerhl

[rchiodo]

I don’t believe it supports an or condition. You’ll have to just change it to array. However the code has to handle old settings so the actual code should be ‘string | string[]’

[janosh]

@joyceerhl I did install node@12 before trying npm ci but still ran into this error:

gyp: Call to 'node -e "require('nan')"' returned exit status 0 while in binding.gyp. while trying to load binding.gyp
gyp ERR! configure error 
gyp ERR! stack Error: `gyp` failed with exit code: 1