Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Travis CI integrated #29

Merged
merged 2 commits into from
Jun 12, 2019
Merged

Travis CI integrated #29

merged 2 commits into from
Jun 12, 2019

Conversation

ShivangiSingh17
Copy link
Member

Fixed #14
Changes: Added travis.yml file to the codebase.

@ShivangiSingh17
Copy link
Member Author

Suggest changes if any.

@abhijitramesh
Copy link
Member

@yashk2000 Can you run the Travis build with this config on you local repository and review this pr

@yashk2000
Copy link
Member

@abhijitramesh sure 👍

Copy link
Member

@yashk2000 yashk2000 left a comment

Choose a reason for hiding this comment

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

Rename the file to .travis.yml

@vchrombie vchrombie requested a review from yashk2000 June 12, 2019 12:34
@vchrombie
Copy link
Member

Thanks for the PR @ShivangiSingh17. 🙂

I did the changes and I am merging this. 🎉
It will be activated after this merge.

@vchrombie vchrombie merged commit 4bab361 into amfoss:master Jun 12, 2019
@aswinshenoy
Copy link
Contributor

aswinshenoy commented Jun 12, 2019

@ShivangiSingh17, @vchrombie - Please parse yml files through YAML Formatter, the merged .travis.yml had basic syntax issues. Please make sure it passes YAML Lint before committing further changes.

YML is a very unforgiving language, it looks deeply into indentation and stuff just like Python. Be sure, and careful :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate Travis CI
5 participants