azavea / majorkirby

Puts cloudformation stacks into motion
Apache License 2.0
1 stars 1 forks source link

Add test suite and initial support for PyPI publishing via Travis CI #7

Closed hectcastro closed 9 years ago

hectcastro commented 9 years ago

This changeset adds a basic unittest suite for this project, along with instructions for executing it locally and inside of a Docker container. In addition, support for Travis CI was added to execute the test suite after linting.

PyPI deployment has not been tested, but it will only be triggered on Git tags.

Attempts to resolve #4 and #5.

sharph commented 9 years ago

I'm a fan of 80 column lines for the linter, and that's what I was aiming for when I wrote it (mostly to make flake8 happy,) but the JEST standard is 100 columns. @ddohler & @notthatbreezy do you have thoughts on this?

notthatbreezy commented 9 years ago

I like 100 columns personally

sharph commented 9 years ago

LGTM, aside from the line length issue

hectcastro commented 9 years ago

Added flake8 configuration file with settings for 100 column length lines. Also reverted the changes that were made to accommodate the default column width setting.

notthatbreezy commented 9 years ago

+1