Skip to content

Added a .travis.yml file.#3

Merged
bmpvieira merged 4 commits intomasterfrom
travis_yml
Oct 4, 2017
Merged

Added a .travis.yml file.#3
bmpvieira merged 4 commits intomasterfrom
travis_yml

Conversation

@StuntsPT
Copy link
Member

@StuntsPT StuntsPT commented Jun 3, 2017

As per the title. Fixes #2 .

@bmpvieira
Copy link
Member

Same problem as other modules, no tests for Travis to run.

@StuntsPT
Copy link
Member Author

Added the linter tests and fixed most presented errors.
I am not sure how to fix the missing error, since it is about an already declared variable inside an exception.
I'm also not sure what the correct way to activate travis on a specific directory of the repository is.

@bmpvieira bmpvieira merged commit 598917d into master Oct 4, 2017
@bmpvieira
Copy link
Member

Good for now. New issue (#4) for tests.

@bmpvieira bmpvieira deleted the travis_yml branch October 4, 2017 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants