feat(git): allow users to specify custom commit messages. also fixed some var name typos (#1086)

Author: test

packages/@vue/cli-test-utils/assertPromptModule.js

@@ -11,7 +11,7 @@ module.exports = async function assertPromptModule (
   opts = {}
 ) {
   // auto fill non-module prompts
-  if (opts.plguinsOnly) {
+  if (opts.pluginsOnly) {
     expectedPrompts.unshift(
       {
         message: 'Please pick a preset',
@@ -40,7 +40,7 @@ module.exports = async function assertPromptModule (
   const creator = new Creator('test', '/', [].concat(module))
   const preset = await creator.promptAndResolvePreset()
 
-  if (opts.plguinsOnly) {
+  if (opts.pluginsOnly) {
     delete preset.useConfigFiles
   }
   expect(preset).toEqual(expectedOptions)

packages/@vue/cli/bin/vue.js

@@ -39,6 +39,7 @@ program
   .option('-p, --preset <presetName>', 'Skip prompts and use saved or remote preset')
   .option('-d, --default', 'Skip prompts and use default preset')
   .option('-i, --inlinePreset <json>', 'Skip prompts and use inline JSON string as preset')
+  .option('-g, --initialCommit <message>', 'Skip prompt and provide initial commit message (when git is available)')
   .option('-m, --packageManager <command>', 'Use specified npm client when installing dependencies')
   .option('-r, --registry <url>', 'Use specified npm registry when installing dependencies (only for npm)')
   .option('-f, --force', 'Overwrite target directory if it exists')

packages/@vue/cli/lib/Creator.js

@@ -79,6 +79,22 @@ module.exports = class Creator {
 
     // clone before mutating
     preset = cloneDeep(preset)
+
+    let initialCommit
+    if (cliOptions.initialCommit) {
+      initialCommit = cliOptions.initialCommit
+    } else {
+      // clear the console and ask for the initial commit message #1086
+      await clearConsole(true)
+      initialCommit = (await inquirer.prompt([{
+        name: 'commitMessage',
+        when: hasGit,
+        type: 'input',
+        message: 'Initial Git commit message:',
+        default: 'init'
+      }])).commitMessage
+    }
+
     // inject core service
     preset.plugins['@vue/cli-service'] = Object.assign({
       projectName: name
@@ -164,7 +180,7 @@ module.exports = class Creator {
         await run('git', ['config', 'user.name', 'test'])
         await run('git', ['config', 'user.email', '[email protected]'])
       }
-      await run(`git commit -m init`)
+      await run('git', ['commit', '-m', initialCommit])
     }
 
     // log instructions

packages/@vue/cli/lib/promptModules/__tests__/babel.spec.js

@@ -23,7 +23,7 @@ test('should pass', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })
 
@@ -49,6 +49,6 @@ test('should not include the plugin if ts is also present', async () => {
     [mockTSModule, moduleToTest],
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })

packages/@vue/cli/lib/promptModules/__tests__/cssPreprocessors.spec.js

@@ -28,6 +28,6 @@ test('CSS pre-processor ', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })

packages/@vue/cli/lib/promptModules/__tests__/e2e.spec.js

@@ -29,7 +29,7 @@ test('cypress', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })
 
@@ -57,6 +57,6 @@ test('nightwatch', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })

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

@@ -37,7 +37,7 @@ test('base', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })
 
@@ -69,7 +69,7 @@ test('airbnb', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })
 
@@ -101,7 +101,7 @@ test('standard', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })
 
@@ -133,6 +133,6 @@ test('prettier', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })

packages/@vue/cli/lib/promptModules/__tests__/pwa.spec.js

@@ -24,6 +24,6 @@ test('pwa', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })

packages/@vue/cli/lib/promptModules/__tests__/router.spec.js

@@ -23,6 +23,6 @@ test('router', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })

packages/@vue/cli/lib/promptModules/__tests__/typescript.spec.js

@@ -48,7 +48,7 @@ test('with TSLint', async () => {
     [moduleToTest, linterModule],
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })
 
@@ -96,6 +96,6 @@ test('with ESLint', async () => {
     [moduleToTest, linterModule],
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })

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

@@ -29,7 +29,7 @@ test('mocha', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })
 
@@ -57,6 +57,6 @@ test('jest', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })

packages/@vue/cli/lib/promptModules/__tests__/vuex.spec.js

@@ -23,6 +23,6 @@ test('vuex', async () => {
     moduleToTest,
     expectedPrompts,
     expectedOptions,
-    { plguinsOnly: true }
+    { pluginsOnly: true }
   )
 })