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

Issues/95 #119

Merged
merged 44 commits into from
Feb 6, 2016
Merged

Issues/95 #119

merged 44 commits into from
Feb 6, 2016

Conversation

jantman
Copy link
Owner

@jantman jantman commented Feb 6, 2016

This converts to boto3.

…, install virtualenv 13.1.2" - this wasn't the problem, it was the tox env

This reverts commit 27fad45.
…now that we have a pair of functions that we just pass an API name into
…ke_with_throttling_retries(); also remove the boto2 ResultSet pagination logic, trimming pagination down ot just dicts
…nd this commit for all accounts I have access to; boto3 conversion and connection refactor should be mostly complete
…n for awslimitchecker.tests.services.result_fixtures.get_boto3_resource_model(), if it can't find local credentials, fires off a HTTP request to the Instance Metadata service. See: #95 (comment)
@codecov-io
Copy link

Current coverage is 100.00%

Merging #119 into develop will not affect coverage as of 88a54f1

@@            develop    #119   diff @@
=======================================
  Files            17      17       
  Stmts          1417    1390    -27
  Branches        224     222     -2
  Methods           0       0       
=======================================
- Hit            1417    1390    -27
  Partial           0       0       
  Missed            0       0       

Review entire Coverage Diff as of 88a54f1

Powered by Codecov. Updated on successful CI builds.

jantman added a commit that referenced this pull request Feb 6, 2016
@jantman jantman merged commit a573b21 into develop Feb 6, 2016
@jantman jantman deleted the issues/95 branch February 6, 2016 00:27
@jantman jantman mentioned this pull request Feb 6, 2016
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