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

Py 2.7/3.6 compatible, enable travis, Sphinx >= 1.4.3 #1

Merged
merged 3 commits into from
Nov 23, 2017

Conversation

joeced
Copy link
Collaborator

@joeced joeced commented Nov 22, 2017

  • I ran 2to3.py and added from future imports. So now its Python 2.7 and 3.6 compatible
  • Setup travis to run the tests and generate html from the docs.
  • Fixed the Sphinx <1.4.3 warning on "WARNING: 4 column based index found"
  • Issues:
    • Still has warnings like "WARNING: while setting up extension sphinxcontrib.matlab: directive 'automodule' is already registered, it will be overridden"

@mikofski
Copy link
Collaborator

Hi @joeced I added travis-ci, if you make another push, it should build. How about adding:

 from builtins import dict

to make it efficient on both python2 and 3?

otherwise looks fine to me

@mikofski
Copy link
Collaborator

In setup.py just need to add future to requires then it should pass.

Are you interested in maintaining this domain? I don't use Matlab anymore, so this project needs a new owner. Interested?

@joeced
Copy link
Collaborator Author

joeced commented Nov 23, 2017

Yes I would be interested in maintaining it, even though I know little about the inner workings of Sphinx.

I use Matlab as part of my daily work, and have several projects that benefit from Sphinx-based documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants