#16 Broke python2 support

[wojciechwroblewski]

Thanks to #16 we've got python3 support, which is great, but unfortunately it also broke support for python2. Tested on:

$ python3 --version
Python 3.5.1
$ python2 --version
Python 2.7.11
# python --version
Python 2.7.5

Using sample code provided in README.md:

#!/usr/bin/env python

from PythonConfluenceAPI import ConfluenceAPI

conf = ConfluenceAPI('admin', '12345', 'http://localhost/confluence/')

confluence_recent_data = conf.get_content()
print(len(confluence_recent_data))

Results:

$ python2 test.py
5
5
$ python3 test.py
5

It's definitely something related to future lib, my guess is that ConfluenceAPI object is somehow created twice. I decided to stick with 62c95b8 as I don't really need python3 support.

[rpcope1]

Noted. I'll take a look.

On Tue, Mar 22, 2016 at 4:59 AM, s1gnalrunner notifications@github.com
wrote:

Thanks to #16
#16 we've
got python3 support, which is great, but unfortunately it also broke
support for python2. Tested on:

$ python3 --version
Python 3.5.1
$ python2 --version
Python 2.7.11

python --version

Python 2.7.5

Using sample code provided in README.md:

#!/usr/bin/env python

from PythonConfluenceAPI import ConfluenceAPI

conf = ConfluenceAPI('admin', '12345', 'http://localhost/confluence/')

confluence_recent_data = conf.get_content()
print(len(confluence_recent_data))

Results:

$ python2 test.py
5
5
$ python3 test.py
5

It's definitely something related to future lib, my guess is that
ConfluenceAPI object is somehow created twice. I decided to stick with
62c95b8
62c95b8
as I don't really need python3 support.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#18

[astewart-twist]

+1