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

revert to cli_version as the name change breaks everyone's builds #318

Merged
merged 1 commit into from
Aug 27, 2015

Conversation

kalbasit
Copy link
Contributor

fixes #315

/cc @aj-may

@Joshua-Anderson
Copy link
Contributor

@kalbasit You need to change client_version in the specs as well. 😃

@kalbasit
Copy link
Contributor Author

done, thank you!

@aj-may
Copy link
Contributor

aj-may commented Aug 27, 2015

thats fine. I think client_version is more accurate, but either is cool with me. Let me know when this gets deployed so I can update our travis files to match.

@aj-may
Copy link
Contributor

aj-may commented Aug 27, 2015

Can we support both options?

BanzaiMan added a commit that referenced this pull request Aug 27, 2015
revert to cli_version as the name change breaks everyone's builds
@BanzaiMan BanzaiMan merged commit 9c0f80d into travis-ci:master Aug 27, 2015
@kalbasit kalbasit deleted the issue_315 branch August 27, 2015 18:49
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.

Deis not able to take the cli_version dynamically since #297
5 participants