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

Update to use moab-versioning 2.0 #3

Merged
merged 6 commits into from Feb 12, 2016
Merged

Update to use moab-versioning 2.0 #3

merged 6 commits into from Feb 12, 2016

Conversation

cbeer
Copy link
Contributor

@cbeer cbeer commented Feb 11, 2016

No description provided.

s.add_dependency 'rest-client', '~> 1.6'
s.add_dependency 'moab-versioning', '~> 1.3'
s.add_dependency 'moab-versioning', '~> 2.0'
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't know anything about 2.0, the last release that I'm aware of is 1.4 (I think). So I can't evaluate this without dedicated time to evaluate the 2.0 release.

@dazza-codes
Copy link
Contributor

Looks good to go. Having reviewed the moab 2.x, I'm OK with an upgrade given all the specs are passing.

@dazza-codes
Copy link
Contributor

Something is funky on travis for the ruby 2.1.2 build, everything else is OK. It can't install this gem at 0.6.0. The ruby 2.3.0 works, so that's good to know.

@cbeer
Copy link
Contributor Author

cbeer commented Feb 11, 2016

Ah, yep. I think we've found this build error on other projects on travis.. something about bundler versions, maybe? I'd suggest dropping the 2.1.2 build, 👌 ?

@dazza-codes
Copy link
Contributor

If we drop the 2.1.2 build, should we also upgrade the ruby version on all the sdr-* boxes? It looks like this repo runs OK on 2.2.4, so I wonder if all the SDR-PC components and robot stuff all works on 2.2.4? Or are we feeling lucky today (just drop it for travis and assume it's OK on our boxes)?

@cbeer
Copy link
Contributor Author

cbeer commented Feb 12, 2016

I've applied the fix for older ruby builds, and the build is now passing under ruby 2.1.2.

dazza-codes added a commit that referenced this pull request Feb 12, 2016
Update to use moab-versioning 2.0
@dazza-codes dazza-codes merged commit 76d103b into master Feb 12, 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.

None yet

2 participants