Kami commented on pull request #1476:
URL: https://github.com/apache/libcloud/pull/1476#issuecomment-674096348


   I've added some in-line comments.
   
   Overall, the changes mostly look good, but the code needs a bit more work to 
reduce duplication and to make sure the methods and method arguments comply 
with the base Libcloud API - that's one of the main ideas behind Libcloud - all 
the provider drivers should expose the same API (where possible), so users can 
easily switch between different providers and non-standard methods and 
arguments should be prefixed with ``ex_``.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to