Skip to content
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

removed region #20

Merged
merged 2 commits into from
Aug 6, 2020
Merged

removed region #20

merged 2 commits into from
Aug 6, 2020

Conversation

tgallagher91
Copy link
Contributor

removed region variable and arguments from s3 module to resolve issue #19

removed region variable and arguments from s3 module to resolve issue #19
Copy link

@Jakeasaurus Jakeasaurus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just delete the lines rather than commenting out please.

completely deleted instead of comment
Copy link
Contributor Author

@tgallagher91 tgallagher91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did it

Copy link

@Jakeasaurus Jakeasaurus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved.

@Jakeasaurus Jakeasaurus merged commit ec9f395 into master Aug 6, 2020
@Jakeasaurus Jakeasaurus deleted the dev_fix_issue_19 branch August 6, 2020 17:39
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.

2 participants