8000 Basic Travis CI Support by t8y8 · Pull Request #16 · tableau/document-api-python · GitHub
[go: up one dir, main page]

Skip to content

Basic Travis CI Support #16

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

Merged
merged 1 commit into from
Jun 17, 2016
Merged

Basic Travis CI Support #16

merged 1 commit into from
Jun 17, 2016

Conversation

t8y8
Copy link
Contributor
@t8y8 t8y8 commented Jun 17, 2016

Basic TravisCI yml file that tests the supported versions, runs PEP8 and runs the samples.

This doesn't have anything configured around branches or notification settings.

I also fixed a small pep8 violation in init.py and added .DS_STORE to the ignore file because my mac is evil

… and runs the samples.

This doesn't have anything configured around branches or notification settings.
@t8y8
Copy link
Contributor Author
t8y8 commented Jun 17, 2016

@RussTheAerialist per our convo, here ya go.

https://travis-ci.org/t8y8/document-api-python/builds/138257513 shows the build example

# Tests
- python test.py
# pep8
- pep8 --ignore=E501 .
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E501 is the 79 character line length requirement

@graysonarts graysonarts merged commit d600124 into tableau:master Jun 17, 2016
@graysonarts
Copy link
Contributor

Awesome! Ben and I will turn on travis for master sometime today.

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
0