Pep8 lint

[cg-cnu]

Fixes #81

[mottosso]

Thanks for this @cg-cnu. I'm getting a slightly different set of warnings here from my linter (Anaconda, Sublime 3) so I think it's a good opportunity to add linting to the automated test suite here on GitHub.

I'll take a look at that within the coming days.

[cg-cnu]

@mottosso Sounds fantastic! Let me know if I can be of any help! 😄

[mottosso]

Actually, if you'd like, I'd be happy for you to do that. :)

It'd involve setting up Travis, running docker-maya on it, installing and running pylint on it to check for errors. It's a challenge. Let me know. :)

[cg-cnu]

Wow! Thanks for the offer 🙇 I would be happy to pickup. 😄
I have setup jenkins before but never tried travis and briefly played with docker. So, might need some guidance along the way 🙂

[mottosso]

Certainly! Have a look at the Qt.py project for inspiration; I'd imagine the set-up to be quite similar.

• Qt.py/.travis.yml

Notice that I've linked to an older commit, when the Dockerfile and Travis configuration file wasn't so complex. In a nutshell, you can start by pulling mottosso/docker-maya and get the pylint working in there. You'll need to run it via mayapy, initialise Maya and then run it, such that it can gain access to members of maya.cmds and friends.

More inspiration can be found in the Avalon project, which uses the docker-maya project as well.

• Avalon/.travis.yml
• Avalon/Dockerfile

In this Dockerfile, you can see how it builds upon docker-maya, and adds a custom command to it. In the .travis.yml file, you can see how it's being used. But you can focus on getting a Dockerfile up and running first, and having it run successfully on your local machine first. Once that's done, getting Travis to run it is really straightforward.

Any questions, fire away. :)