chore: cleanup comments

Author: yyx990803

packages/@vue/cli-plugin-unit-jest/index.js

@@ -14,7 +14,6 @@ module.exports = api => {
     process.env.VUE_CLI_BABEL_TARGET_NODE = true
     process.env.VUE_CLI_BABEL_TRANSPILE_MODULES = true
 
-    // TODO execa jest w/ --config jest.config.js
     const execa = require('execa')
     const jestBinPath = require.resolve('jest/bin/jest')
 

packages/@vue/cli-service/lib/commands/build.js

@@ -1,5 +1,3 @@
-// TODO handle alternative build targets
-
 const defaults = {
   mode: 'production',
   target: 'app'
@@ -10,8 +8,9 @@ module.exports = (api, options) => {
     description: 'build for production',
     usage: 'vue-cli-service build [options]',
     options: {
-      '--mode': `specify env mode (default: ${defaults.mode})`,
-      '--target': `app | library | web-component (default: ${defaults.target})`
+      // TODO
+      // '--mode': `specify env mode (default: ${defaults.mode})`,
+      // '--target': `app | library | web-component (default: ${defaults.target})`
     }
   }, args => {
     api.setMode(args.mode || defaults.mode)
@@ -46,8 +45,7 @@ module.exports = (api, options) => {
             process.stdout.write(stats.toString({
               colors: true,
               modules: false,
-              // TODO set this to true if using TS
-              children: false,
+              children: api.hasPlugin('typescript'),
               chunks: false,
               chunkModules: false
             }) + '\n\n')

packages/@vue/cli-service/lib/commands/serve.js

@@ -108,7 +108,6 @@ module.exports = (api, options) => {
           } else {
             console.log(`  App is served in production mode.`)
             console.log(`  Note this is for preview or E2E testing only.`)
-            // TODO console.log(`  See URL for more deployment tips.`)
           }
           console.log()
 

packages/@vue/cli-service/lib/config/css.js

@@ -80,6 +80,7 @@ module.exports = (api, options) => {
           }, userOptions)])
     }
 
-    // TODO document receipe for adding sass-resource-loader
+    // TODO document receipe for using css.loaderOptions to add `data` option
+    // to sass-loader
   })
 }

packages/@vue/cli-service/lib/config/prod.js

@@ -121,7 +121,7 @@ module.exports = (api, options) => {
             ignore: ['index.html', '.*']
           }]])
 
-      // TODO investigate DLL plugin options
+      // TODO investigate happypack/cache-loader/DLL plugin options
     }
   })
 }

packages/@vue/cli/lib/promptModules/linter.js

@@ -5,8 +5,7 @@ module.exports = cli => {
   cli.injectFeature({
     name: 'Linter / Formatter',
     value: 'linter',
-    short: 'Linter',
-    checked: true
+    short: 'Linter'
   })
 
   cli.injectPrompt({
@@ -49,13 +48,11 @@ module.exports = cli => {
     choices: [
       {
         name: 'Lint on save',
-        value: 'save',
-        checked: true
+        value: 'save'
       },
       {
         name: 'Lint and fix on commit' + (hasGit ? '' : chalk.red(' (requires Git)')),
-        value: 'commit',
-        checked: true
+        value: 'commit'
       }
     ]
   })

packages/@vue/cli/lib/promptModules/unit.js

@@ -2,8 +2,7 @@ module.exports = cli => {
   cli.injectFeature({
     name: 'Unit Testing',
     value: 'unit',
-    short: 'Unit',
-    checked: true
+    short: 'Unit'
   })
 
   cli.injectPrompt({