search

ga-wdi-boston / aws-s3-setup-guide

Other
3 stars 108 forks source link

015/master #11

Closed BenGitsCode closed 7 years ago

BenGitsCode commented 7 years ago

Addresses #7 and #9

10 and #4 resolutions are unclear to me. @gaand thoughts?

gaand commented 7 years ago

I am also unclear on #10. I believe #4 should be closed.

BenGitsCode commented 7 years ago

Just saw the merge :( You were right to merge, I should have followed my instinct and not accommodated the issue in the first place, but I'd like to talk about this handholding balance at some point @gaand maybe with @jrhorn424 and @payne-chris-r as well.

I can understand wanting emphasis here because ignoring instructions can result in real financial loss, but all instructions should be followed with that level of scrutiny and sense of importance. I know there's a time and place for hand-holding, but I don't think this was it.