Merge pull request #272 from Cox-Automotive/bugfix

fixes a bug with printing the name of the generated profile
Cox-Automotive · Nov 9, 2023 · 820d04c · 820d04c
2 parents 238dd42 + 63c4e40
commit 820d04c
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 12 deletions.
diff --git a/changelog.txt b/changelog.txt
@@ -3,7 +3,7 @@
 
 Thanks for upgrading to the latest version of the ALKS CLI!
 
-* Adds commands for managing AWS profiles
+* Fixes the feedback message when generating a single profile
 
 Have feedback? https://github.com/Cox-Automotive/ALKS-CLI/issues
 

diff --git a/dist/changelog.txt b/dist/changelog.txt
@@ -3,7 +3,7 @@
 
 Thanks for upgrading to the latest version of the ALKS CLI!
 
-* Adds commands for managing AWS profiles
+* Fixes the feedback message when generating a single profile
 
 Have feedback? https://github.com/Cox-Automotive/ALKS-CLI/issues
 

diff --git a/dist/src/lib/handlers/alks-profiles-generate.js b/dist/src/lib/handlers/alks-profiles-generate.js
diff --git a/dist/src/lib/handlers/alks-profiles-generate.js.map b/dist/src/lib/handlers/alks-profiles-generate.js.map
diff --git a/src/lib/handlers/alks-profiles-generate.ts b/src/lib/handlers/alks-profiles-generate.ts
@@ -44,14 +44,11 @@ export async function handleAlksProfilesGenerate(
       throw new Error('role is required');
     }
 
-    generateProfile(
-      options.account,
-      options.role,
-      options.profile ?? options.namedProfile,
-      options.force
-    );
+    const profileName = options.profile ?? options.namedProfile;
+
+    generateProfile(options.account, options.role, profileName, options.force);
 
-    console.error(`Profile ${options.profile} generated`);
+    console.error(`Profile ${profileName} generated`);
   } else {
     throw new Error(
       'Either --all or --account and --role is required at a minimum'