aws
Changing sudo pip to pip in readme.rst file.
Codecov Report
Merging #4224 into develop will not change coverage. The diff coverage is
n/a.
@@ Coverage Diff @@
## develop #4224 +/- ##
========================================
Coverage 94.41% 94.41%
========================================
Files 188 188
Lines 14151 14151
========================================
Hits 13360 13360
Misses 791 791
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 069c7f9...3b083ef. Read the comment docs.
Hi @kshithijiyer,
Apologies for the delay. We're cleaning up our backlog according to our new contribution process. I understand the sentiment of this change, and agree with @merwok that just dropping the sudo is probably not the best. I would possibly just drop this command entirely and rely on suggesting that users use a virtual environment. The logic here is that's what the AWS CLI v1 installation guide also suggests that, as well as a --user install. It does not direct users to install system-wide without using an operating system's package manager.
I think we should draw a better distinction between installing for development purposes here in the GitHub repository, and direct end users more towards the content in the Installation section of the AWS CLI User Guide.
Given the lag between opening this and now, I'm happy to make these changes, but I can leave it for you to do as well since it's a minimal change from what you already have. It might be better for you to open a new PR given the conflicts present from the long time; otherwise you'll need to rebase as well.
Let me know how you'd like to proceed!
After some further discussion, we'd be even happier to remove the entire installation section, save for things that a developer would use, and redirect all end users to the User Guide installation section.
Given that we haven't heard back on this PR, I'm going to close it due to inactivity. As suggested above, there are broader improvements to the installation documentation that could be made here and I think that can be in a separate PR.
