-
Notifications
You must be signed in to change notification settings - Fork 4k
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
fix(cli): wrongly assume aws config file always exists #6001
Conversation
Signed-off-by: Duarte Nunes <[email protected]>
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
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.
Oouch.. Thanks for this. I am curious - did this cause any issues? Is there a specific bug this would fix?
I just happen to notice it. I guess it depends on the behavior of the sdk, but I’d imagine it causes an enoent somewhere. |
Thank you for contributing! Your pull request is now being automatically merged. |
1 similar comment
Thank you for contributing! Your pull request is now being automatically merged. |
The base fork belongs to an organization and can't be updated by maintainers (mergify in this case). I've created a mirror PR from a personal base here: #6196 |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Signed-off-by: Duarte Nunes [email protected]
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license