🏗🔧：reconfigure vscode for smoother startup

.devcontainer/devcontainer.json

@@ -1,28 +1,32 @@
 // For format details, see https://aka.ms/devcontainer.json. For config options,
 // see documentation located at the following URL:
 // https://github.com/OpenINF/docker-fisher/tree/main/library-scripts/docs
-// For format details, see https://aka.ms/devcontainer.json. For config options, see the
-// README at: https://github.com/devcontainers/templates/tree/main/src/docker-outside-of-docker
+// For config options, see the README at:
+// https://github.com/devcontainers/templates/tree/main/src/docker-outside-of-docker
 {
-	"name": "Docker outside of Docker",
-	// Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
-    "image": "mcr.microsoft.com/devcontainers/javascript-node:18-bullseye"
+  "name": "Docker outside of Docker",
+  // Or use a Dockerfile or Docker Compose file.
+  // TODO(DerekNonGeneric): update comment to include link to repo
+  "image": "docker.io/openinf/centos7-fisher:latest",
 
-	"features": {
-		"ghcr.io/devcontainers/features/docker-outside-of-docker:1": {
-			"version": "latest",
-			"enableNonRootDocker": "true",
-			"moby": "false"
-		}
-	},
+// TODO(): remove this whole feature once we have replicated it
+  "features": {
+    "ghcr.io/devcontainers/features/docker-outside-of-docker:1": {
+      "version": "latest",
+      "enableNonRootDocker": "true",
+      "moby": "false"
+    }
+  },
 
-	// Use this environment variable if you need to bind mount your local source code into a new container.
-	"remoteEnv": {
-		"LOCAL_WORKSPACE_FOLDER": "${localWorkspaceFolder}",
-		"PATH": "${containerEnv:PATH}:~/.rbenv/shims/"
-	}
+  // Use this environment variable if you need to bind mount your local source
+  // code into a new container.
+  "remoteEnv": {
+    "LOCAL_WORKSPACE_FOLDER": "${localWorkspaceFolder}",
+    "PATH": "${containerEnv:PATH}:~/.rbenv/shims/"
+  },
 
-	// Use 'forwardPorts' to make a list of ports inside the container available locally.
+  // Use 'forwardPorts' to make a list of ports inside the container available
+  // locally.
   "forwardPorts": [
     // Jekyll server
     4000,
@@ -35,9 +39,10 @@
   // Use 'postCreateCommand' to run commands after the container is created.
   "postCreateCommand": "fish .devcontainer/post-create.fish",
 
-	// Configure tool-specific properties.
-	// "customizations": {},
+  // Configure tool-specific properties.
+  // "customizations": {},
 
-	// Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
-	// "remoteUser": "root"
+  // Uncomment to connect as root instead.
+  // More info: https://aka.ms/dev-containers-non-root.
+  "remoteUser": "node"
 }

.gitattributes

@@ -1,9 +1,9 @@
 *                       text=auto eol=lf linguist-detectable
 .*.jsonc                diff=jsonc linguist-language=jsonc
 .*.json5                diff=json5 linguist-language=json5
-.devcontainer/*.json    diff=jsonc linguist-language=jsonc
 .eslintrc.json          diff=jsonc linguist-language=jsonc
 .snyk                   diff=yaml linguist-language=yaml
+.devcontainer/*.json    diff=jsonc linguist-language=jsonc
 .vscode/*.json          diff=jsonc linguist-language=jsonc
 build/**                linguist-generated=false
 doc/**                  linguist-documentation

.vscode/extensions.json

@@ -8,7 +8,7 @@
     // https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint
     "dbaeumer.vscode-eslint",
     // https://marketplace.visualstudio.com/items?itemName=dprint.dprint
-    "dprint.dprint",
+    // "dprint.dprint",
     // https://marketplace.visualstudio.com/items?itemName=editorconfig.editorconfig
     "editorconfig.editorconfig",
     // https://marketplace.visualstudio.com/items?itemName=esbenp.prettier-vscode

.vscode/settings.json

@@ -7,7 +7,6 @@
 
   "editor.stickyScroll.enabled": false,
   "editor.codeLens": true,
-  "eslint.format.enable": true,
   "editor.formatOnSave": true,
 
   "[javascript][json][json5][jsonc][typescript][yaml]": {
@@ -21,12 +20,12 @@
     "editor.defaultFormatter": "Shopify.theme-check-vscode"
   },
 
-  "[toml]": {
-    "editor.codeActionsOnSave": {
-      "source.fixAll.dprint": "explicit"
-    },
-    "editor.defaultFormatter": "dprint.dprint"
-  },
+  // "[toml]": {
+  //   "editor.codeActionsOnSave": {
+  //     "source.fixAll.dprint": "explicit"
+  //   },
+  //   "editor.defaultFormatter": "dprint.dprint"
+  // },
 
   "[markdown]": {
     "editor.codeActionsOnSave": {
@@ -44,8 +43,9 @@
   // Terminal
   //////////////////////////////////////////////////////////////////////////////
 
-  "terminal.integrated.useWslProfiles": true,
-
+  // A set of terminal profile customizations for Linux that allows adding,
+  // removing, or changing how terminals are launched. Profiles are made up of
+  // a mandatory path, optional arguments, and other presentation options.
   "terminal.integrated.profiles.linux": {
     "bash": {
       "path": "bash"
@@ -63,10 +63,62 @@
     "pwsh": null
   },
 
-  // "terminal.integrated.defaultProfile.linux": "bash",
+  // The default terminal profile on Linux.
+  "terminal.integrated.defaultProfile.linux": "fish",
 
+  // TODO(DerekNonGeneric):
+  // A set of terminal profile customizations for macOS which allows adding,
+  // removing or changing how terminals are launched. Profiles are made up of
+  // a mandatory path, optional arguments and other presentation options.
+  "terminal.integrated.profiles.osx": {},
+
+  // TODO(DerekNonGeneric): The default terminal profile on macOS.
+  "terminal.integrated.defaultProfile.osx": null,
+
+  // Controls whether or not WSL distros are shown in the terminal dropdown.
+  // Not actually recommended, but useful one-offs if that may be the case.
+  "terminal.integrated.useWslProfiles": true,
+
+  // A set of terminal profile customizations for Windows which allows adding,
+  // removing or changing how terminals are launched. Profiles are made up of
+  // a mandatory path, optional arguments and other presentation options.
+  "terminal.integrated.profiles.windows": {
+    "Git Bash": null,
+    "pwsh": {
+      "path": "pwsh.exe",
+      "source": "PowerShell",
+      "icon": "terminal-powershell",
+    },
+    "Command Prompt": {
+      "path": [
+        "${env:windir}\\Sysnative\\cmd.exe",
+        "${env:windir}\\System32\\cmd.exe",
+      ],
+      "icon": "terminal-cmd",
+    },
+  },
+
+  // The default terminal profile on Windows.
+  "terminal.integrated.defaultProfile.windows": "pwsh",
+
+  // Controls whether the terminal will ignore bracketed paste mode even if the
+  // terminal was put into the mode, omitting the `\x1b[200~` and `\x1b[201~`
+  // sequences when pasting. This is useful when the shell is not respecting the
+  // mode which can happen in sub-shells for example.
+  "terminal.integrated.ignoreBracketedPasteMode": true,
+
+  // Controls the working directory a split terminal starts with.
+  //  - workspaceRoot: A new split terminal will use the workspace root as the
+  //    working directory. In a multi-root workspace a choice for which root
+  //    folder to use is offered.
+  //  - initial: A new split terminal will use the working directory that the
+  //    parent terminal started with.
+  //  - inherited: On macOS and Linux, a new split terminal will use the working
+  //    directory of the parent terminal. On Windows, this behaves the same as
+  //    initial.
   "terminal.integrated.splitCwd": "workspaceRoot",
 
+  // Automatic script injection has been disabled.
   "terminal.integrated.shellIntegration.enabled": false,
 
   // Files
@@ -109,7 +161,6 @@
   // This setting will control whether multiline comments are styled using the
   // annotation tags. If false, multiline comments will be presented w/o decor.
   "better-comments.multilineComments": true,
-
   "better-comments.tags": [
     {
       "tag": "//",
@@ -118,20 +169,22 @@
   ],
 
   // Dprint
-  "dprint.path": "~/.dprint/bin/dprint",
+  /////////////////////////////////////////////
+
+  // "dprint.path": "~/.dprint/bin/dprint",
 
   // EditorConfig (https://editorconfig.org/)
   /////////////////////////////////////////////
 
   // The following settings should be inherited from the .editorconfig file and
   // need not be re-specified in this file: files.encoding, files.eol,
   // files.insertFinalNewline, and files.trimTrailingWhitespace.
-
   "editorconfig.generateAuto": false,
 
   // ESLint (ESLint is a separate extension)
   /////////////////////////////////////////////
 
+  "eslint.format.enable": true,
   "eslint.validate": [
     "javascript",
     "json",
@@ -141,7 +194,6 @@
     "typescript",
     "yaml"
   ],
-
   "eslint.workingDirectories": [
     "./"
   ],
@@ -158,3 +210,4 @@
     }
   }
 }
+