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-ci #12

Closed
dynamitechetan opened this issue Aug 3, 2016 · 2 comments
Closed

travis-ci #12

dynamitechetan opened this issue Aug 3, 2016 · 2 comments

Comments

@dynamitechetan
Copy link
Member

@h-darji why you removed python version 2.7 from travis.yml file? Many people are still using python ver 2.7.

@harshildarji
Copy link
Member

harshildarji commented Aug 3, 2016

I removed it from .travis.yml file, because it is causing build to fail when we push or merge (reason is some of the python scripts are written in python > 3 and dose not support python < 3). It does not affect actual code, when I make all the scripts to run in all python versions, I'll add 2.7 back to .travis.yml.

@AnupKumarPanwar
Copy link
Member

Yes even I think that some merge requests are in different versions and merging such requests may lead to spoiling of whole code.. And we will land back to the issue of non-pythonic code

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

No branches or pull requests

3 participants