Skip to content
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

travis isn't useful yet #18

Closed
CoXier opened this issue Apr 24, 2017 · 6 comments
Closed

travis isn't useful yet #18

CoXier opened this issue Apr 24, 2017 · 6 comments
Assignees
Labels

Comments

@CoXier
Copy link
Contributor

CoXier commented Apr 24, 2017

As I see,there isn't any test in Matisse .So why do you use travis.

@unixzii
Copy link
Contributor

unixzii commented Apr 24, 2017

Ah, maybe they just want to keep testing whether it can pass the compilation. LOL

@CoXier
Copy link
Contributor Author

CoXier commented Apr 24, 2017

@unixzii
It seems to make sense.LOL

@CoXier CoXier closed this as completed Apr 24, 2017
@gejiaheng
Copy link
Contributor

@CoXier @unixzii
Thanks for reporting this. Actually this is my first time using Travis and I haven't completely set it up. I'll see whether Travis can be somehow useful without any tests. Or I'll turn it off temporarily.

@gejiaheng gejiaheng reopened this Apr 25, 2017
@gejiaheng gejiaheng self-assigned this Apr 25, 2017
@gejiaheng gejiaheng added the bug label Apr 25, 2017
@unixzii
Copy link
Contributor

unixzii commented Apr 25, 2017

@gejiaheng Yep, Travis CI can be very useful when you want to achieve automation, not only stuff like unit testing, but also releasing.

Let's say you want to publish a test to fir.im whenever you push a commit on specific branch, you can literally write a Gradle script, but instead of doing it on your local machine, Travis can do it for you on its own docker environment, which is pretty handy.

@gejiaheng
Copy link
Contributor

@unixzii
Sounds awesome, thanks for enlightening me!

@CoXier
Copy link
Contributor Author

CoXier commented May 8, 2017

Now CheckStyle is added in travis. see pr

@CoXier CoXier closed this as completed May 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants