search

forem / selfhost

Selfhost your Forem Community on your own infrastructure 🎉
https://www.forem.com/get-started/
GNU Affero General Public License v3.0
1.45k stars 175 forks source link

Aws.yml file updated #88

Closed ikarus-sagar closed 1 year ago

ikarus-sagar commented 1 year ago

Hey There I removed Un necessary Prints and Corrected it as Required , all the aws module are now updated. Please take look and now It Works Perfectly as it should.

J-Drakon commented 1 year ago

Hey There I removed Un necessary Prints and Corrected it as Required , all the aws module are now updated. Please take look and now It Works Perfectly as it should.

Thanks.. this Works now.

klardotsh commented 1 year ago

Hey @ikarus-sagar, I missed this, sorry about that. Just so you know for the future, you don't have to close and re-open a new PR to address feedback, it's alright to amend and force-push your updated work to the existing PR branch.

I should have some time to review this tomorrow morning, thanks for the (re-)submission!

ikarus-sagar commented 1 year ago

Hey @ikarus-sagar, I missed this, sorry about that. Just so you know for the future, you don't have to close and re-open a new PR to address feedback, it's alright to amend and force-push your updated work to the existing PR branch.

I should have some time to review this tomorrow morning, thanks for the (re-)submission!

oh ok i will keep that in mind and thanks