New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor(CLI): Modern logs for config
-related command
#10038
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10038 +/- ##
==========================================
- Coverage 85.88% 85.75% -0.14%
==========================================
Files 333 333
Lines 13416 13432 +16
==========================================
- Hits 11523 11518 -5
- Misses 1893 1914 +21
Continue to review full report at Codecov.
|
lib/plugins/aws/configCredentials.js
Outdated
`Failed! ~/.aws/credentials already has a "${this.options.profile}" profile. Use the overwrite flag ("-o" or "--overwrite") to force the update` | ||
); | ||
log.notice.skip( | ||
`Profile "${this.options.profile}" is already configured in ~/.aws/credentials. Use the overwrite flag ("-o" or "--overwrite") to force the update.` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This feels more as an error. I would I think convert it to ServerlessError
exception
lib/plugins/config.js
Outdated
@@ -80,7 +81,9 @@ class Config { | |||
} | |||
if (this.options.autoupdate) { | |||
if (config.get('autoUpdate.enabled')) { | |||
this.serverless.cli.log('Auto update is already turned on'); | |||
const message = 'Auto update is already turned on'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's not generalize to vars
678db89
to
3d667ab
Compare
13632a2
to
09b979a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great 👍
Addresses: #9860