Infoblox-API-Python icon indicating copy to clipboard operation
Infoblox-API-Python copied to clipboard

Added new method, PEP8

Open kraigu opened this issue 10 years ago • 2 comments

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?

kraigu avatar Oct 13 '15 13:10 kraigu

Send me the code. I'll review and put it on master branch.

  From: Mike <[email protected]>

To: Infoblox-Development/Infoblox-API-Python [email protected] 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.

igor-feoktistov avatar Oct 13 '15 15:10 igor-feoktistov

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 [email protected] wrote:

Send me the code. I'll review and put it on master branch.

From: Mike [email protected] To: Infoblox-Development/Infoblox-API-Python [email protected] 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.

kraigu avatar Oct 13 '15 15:10 kraigu