Skip to content

Conversation

Br3nda
Copy link
Contributor

@Br3nda Br3nda commented Mar 11, 2016

Added Travis.yml, to use free continuous integration.

Enum was added in python3.4, so only tests in that or later.

Br3nda added 3 commits March 11, 2016 02:01
many of these won't pass, but let's see how many do
Mostly for Travis-ci to use
@Br3nda
Copy link
Contributor Author

Br3nda commented Mar 11, 2016

Build Status

@MartinHjelmare
Copy link
Collaborator

Nice! @theolind has to decide if Travis testing should be activated for this repo. We could add the travis config anyhow, for fork owners, but best value would be to use it here in the original repo.

Have you tested the travis build in your fork? Edit: Nevermind, I see the badge now. 🙈 👍

# command to install dependencies
install: "pip install -r requirements.txt"
# command to run tests
script: nosetests
Copy link
Collaborator

Choose a reason for hiding this comment

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

An empty line in the end of the file is missing.

@Br3nda
Copy link
Contributor Author

Br3nda commented Mar 12, 2016

Dev seems older, only because you've got "version='0.5'," in master and version='0.4' in dev.
I can make a new PR to dev branch.

@MartinHjelmare
Copy link
Collaborator

Yeah, we haven't been consistent with version bumps and back merging. Best option now is to base your PR off master but direct the PR to dev.

@MartinHjelmare
Copy link
Collaborator

Closing this one. See #31 instead.

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