Open kraigu opened 8 years ago
Send me the code. I'll review and put it on master branch.
From: Mike <notifications@github.com>
To: Infoblox-Development/Infoblox-API-Python Infoblox-API-Python@noreply.github.com Sent: Tuesday, October 13, 2015 6:13 AM Subject: [Infoblox-API-Python] Added new method, PEP8 (#9)
I've made a series of commits to my fork that make the code PEP8 compliant, and which add a new method get_all_networks(). Wasn't sure how best to send you a pull request, cherrypicking would be difficult because I suck at being other-dev friendly in git, but I can just send you one and you can trust me, or have a look yourself?— Reply to this email directly or view it on GitHub.
I have it all in my fork at: https://github.com/kraigu/Infoblox-API-Python
One problem with get_all_networks() is it only gets all v4 networks. :|
Mike
On 13 Oct 2015, at 11:15, Igor Feoktistov notifications@github.com wrote:
Send me the code. I'll review and put it on master branch.
From: Mike notifications@github.com To: Infoblox-Development/Infoblox-API-Python Infoblox-API-Python@noreply.github.com Sent: Tuesday, October 13, 2015 6:13 AM Subject: [Infoblox-API-Python] Added new method, PEP8 (#9)
I've made a series of commits to my fork that make the code PEP8 compliant, and which add a new method get_all_networks(). Wasn't sure how best to send you a pull request, cherrypicking would be difficult because I suck at being other-dev friendly in git, but I can just send you one and you can trust me, or have a look yourself?— Reply to this email directly or view it on GitHub.
— Reply to this email directly or view it on GitHub.
I've made a series of commits to my fork that make the code PEP8 compliant, and which add a new method get_all_networks(). Wasn't sure how best to send you a pull request, cherrypicking would be difficult because I suck at being other-dev friendly in git, but I can just send you one and you can trust me, or have a look yourself?