feat(vue-cli): Choosing to save as a preset tells you where it is sav…

…ed (#5212) close #5197 Co-authored-by: Jair Reina <Jair.Reina@digitaslbi.com>
vuejs · Mar 21, 2020 · e5449ad · e5449ad
1 parent c8e6450
commit e5449ad
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/packages/@vue/cli/lib/Creator.js b/packages/@vue/cli/lib/Creator.js
@@ -21,7 +21,8 @@ const {
   saveOptions,
   loadOptions,
   savePreset,
-  validatePreset
+  validatePreset,
+  rcPath
 } = require('./options')
 
 const {
@@ -309,8 +310,9 @@ module.exports = class Creator extends EventEmitter {
     validatePreset(preset)
 
     // save preset
-    if (answers.save && answers.saveName) {
-      savePreset(answers.saveName, preset)
+    if (answers.save && answers.saveName && savePreset(answers.saveName, preset)) {
+      log()
+      log(`🎉  Preset ${chalk.yellow(answers.saveName)} saved in ${chalk.yellow(rcPath)}`)
     }
 
     debug('vue-cli:preset')(preset)

diff --git a/packages/@vue/cli/lib/options.js b/packages/@vue/cli/lib/options.js
@@ -94,6 +94,7 @@ exports.saveOptions = toSave => {
   cachedOptions = options
   try {
     fs.writeFileSync(rcPath, JSON.stringify(options, null, 2))
+    return true
   } catch (e) {
     error(
       `Error saving preferences: ` +
@@ -106,5 +107,5 @@ exports.saveOptions = toSave => {
 exports.savePreset = (name, preset) => {
   const presets = cloneDeep(exports.loadOptions().presets || {})
   presets[name] = preset
-  exports.saveOptions({ presets })
+  return exports.saveOptions({ presets })
 }