Skip to content

We need to prevent crashing when there is no .aws folder. We should only create the folder if an actual command that needs it is called though - #8

Open
sweep-ai-deprecated[bot] wants to merge 2 commits into
masterfrom
we_need_to_prevent_crashing_when_there_is_no_aws
Open

We need to prevent crashing when there is no .aws folder. We should only create the folder if an actual command that needs it is called though#8
sweep-ai-deprecated[bot] wants to merge 2 commits into
masterfrom
we_need_to_prevent_crashing_when_there_is_no_aws

Conversation

@sweep-ai-deprecated

Copy link
Copy Markdown
Contributor

Summary:
To address the issue, we'll add a check to ensure the .aws folder exists before executing any command that requires it. If the folder doesn't exist, we'll create it. This change will prevent the application from crashing when the .aws folder is not present.

To checkout this PR branch, run the following command in your terminal:

git checkout we_need_to_prevent_crashing_when_there_is_no_aws

@jld-adriano

Copy link
Copy Markdown
Contributor

Sweep: You removed the program definition, and you added way too many ensure calls, and they are outside the correct functions

@jld-adriano

Copy link
Copy Markdown
Contributor

Sweep: can you add a log when the folder is being created, to make sure the user knows about the action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant