IMP: removes awscli from requirements.txt

[cduvallet]

For some reason, when sphinx is the first requirement listed then pip install -r requirements.txt runs through an exhaustive search of awscli versions which takes a really long time. Swapping the order fixes the issue.

Tested:

pip install -r requirements.txt
make dirhtml

Which worked!

[cduvallet]

Though now that I think about it, you probably only need awscli to actually deploy the changes, right? So as a developer trying to test my changes to the documentation, all I really need is sphinx right? So maybe it's better to update the developer docs with just that, or make a requirements.txt for deployers separate from contributors?

[gregcaporaso]

Thanks for the PR @cduvallet, and sorry for the radio silence. We're re-working how we're managing issues/PRs to avoid losing track of incoming stuff. @lizgehret will take care of this and your other dev-docs PR.

[lizgehret]

Hey @cduvallet! I agree with you re: only needing sphinx as a developer testing changes locally. I'm happy to update the dev docs with separate instructions for development vs. deployment, unless you'd like to?

[cduvallet]

Awesome! Please go ahead and make the updates 😸

[lizgehret]

Okay, for posterity - I removed awscli from requirements.txt and added that into the environment creation for dev-docs deployment (in cap-lab-secrets repo).

I'll go ahead and get this merged, thanks again @cduvallet!