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
Libcloud 574 routes and route tables #313
Libcloud 574 routes and route tables #313
Conversation
…ute_tables method
@Kami is there anything else I should do here ? |
:type id: ``str`` | ||
|
||
:param routes: A list of routes in the route table. | ||
:type routes: ``list`` of :class:`EC2Route` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add .
before the classes which are located in the same module. In this case:
:type routes: ``list`` of :class:`.EC2Route`
Ok, reviewed the whole pull request. There are some minor issues left, but it mostly looks good. Next step would be adding tests for this new functionality. Let us know if you need help with that. |
@zerthimon Let me know once those issues have been addressed and I will get it merged into trunk. |
…o work with objects instead of ID strings
Fixed issues that you found during code review. |
Merged into trunk. Thanks. |
Support for EC2 Route Tables and Routes.