Skip to content

Travis integration (Add .travis.yml) #9

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 2 commits into from
Aug 2, 2016

Conversation

SergeyTsaplin
Copy link
Contributor

Add travis-ci integration (Need to be additionally configured via https://travis-ci.org)

@SergeyTsaplin
Copy link
Contributor Author

Example of build result https://travis-ci.org/SergeyTsaplin/Python

@harshildarji
Copy link
Member

@SergeyTsaplin
Hi, I've read some documents about travis-ci, but I'm still not able to understand that what it actually do.
Can you please explain it (of course in short)?

@SergeyTsaplin
Copy link
Contributor Author

SergeyTsaplin commented Aug 2, 2016

travis-ci is regular continuous integration service. It provides test runnings on all required platforms on each commit and pull requests. So you always understand if your code was broken before you merge it to master. Also there are some extra abilities for example test coverage, linters runnings etc

@harshildarji
Copy link
Member

@SergeyTsaplin
Last query, How it works?

@SergeyTsaplin
Copy link
Contributor Author

SergeyTsaplin commented Aug 2, 2016

@h-darji githab calls travis' webhook on each pull request, merge or push, travis fetches the changes and runs the commands declared in .travis-ci.yml file

@harshildarji
Copy link
Member

@SergeyTsaplin As you know, travis requires permission from owners of the organization to access repositories, I've requested owners for this. Once they grants access, I'll merge this.

@SergeyTsaplin
Copy link
Contributor Author

@h-darji Of course you have to grant access to reposito

@SergeyTsaplin SergeyTsaplin reopened this Aug 2, 2016
@dynamitechetan dynamitechetan merged commit ccf062a into TheAlgorithms:master Aug 2, 2016
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.

3 participants