Skip to content
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

Add support for python 3 #313

Closed
5 tasks done
ElDeveloper opened this issue Oct 10, 2014 · 4 comments · Fixed by #508
Closed
5 tasks done

Add support for python 3 #313

ElDeveloper opened this issue Oct 10, 2014 · 4 comments · Fixed by #508
Assignees

Comments

@ElDeveloper
Copy link
Member

ElDeveloper commented Oct 10, 2014

  • setup travis to test for 2 and 3.
  • fix broken tests.
  • change make_emperor.py to use click instead of qcli.
  • remove unnecessary code in format.py and any dependency from qcli.
  • move all_tests.py to click

Any takers?

@josenavas
Copy link
Member

There are multiple issues here... e.g. the last one is quite different. I would take the porting to click if I don't have to take all the other stuff....

@ElDeveloper
Copy link
Member Author

Sorry for that, that's all my bad. If you can take that item alone would
be awesome!!

On (Oct-10-14|13:27), josenavas wrote:

There are multiple issues here... e.g. the last one is quite different. I would take the porting to click if I don't have to take all the other stuff....


Reply to this email directly or view it on GitHub:
#313 (comment)

@ElDeveloper
Copy link
Member Author

@josenavas just assigned this one to you, feel free to just take care of the click item. I will break down into separate issues later.

@josenavas
Copy link
Member

Just a heads up. If we port to click, we need to make sure that the actual interface is the same, so we don't break backwards compatibility.

Also, we need to double check that we can generate some kind of HTML documentation from the click interface.

@ElDeveloper ElDeveloper mentioned this issue May 14, 2016
20 tasks
ElDeveloper added a commit to ElDeveloper/emperor that referenced this issue Jun 14, 2016
ElDeveloper added a commit to ElDeveloper/emperor that referenced this issue Jun 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants