Skip to content

Conversation

rquadling
Copy link
Contributor

@rquadling rquadling commented Sep 11, 2017

Modified json and compiled php.

Resolves #1368

@rquadling rquadling mentioned this pull request Sep 11, 2017
Copy link
Contributor

@kstich kstich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking this on! While I test these, there's a small change required for the error.

"argument": "DBSnapshots[].Status"
},
{
"expected": "DBInstanceNotFound",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be 'DBSnapshotNotFound', as specified here. The waiters-2.json.php compile will also need to be redone.

@codecov-io
Copy link

codecov-io commented Sep 12, 2017

Codecov Report

Merging #1370 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1370   +/-   ##
=========================================
  Coverage     91.88%   91.88%           
  Complexity     2466     2466           
=========================================
  Files           142      142           
  Lines          7529     7529           
=========================================
  Hits           6918     6918           
  Misses          611      611

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5f10172...8369de0. Read the comment docs.

@kstich kstich merged commit 4f74d75 into aws:master Sep 29, 2017
@rquadling rquadling deleted the DBSnapshotWaiters branch September 30, 2017 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants