Skip to content
This repository has been archived by the owner on May 11, 2022. It is now read-only.

Update to moab-versioning 2.x, using fully namespaced Moab classes and modules #160

Merged
merged 1 commit into from
Feb 12, 2016

Conversation

cbeer
Copy link
Contributor

@cbeer cbeer commented Feb 11, 2016

No description provided.

@drh-stanford
Copy link

The builds are failing due to a ruby 2.1 constraint. Our Travis builds are doing 1.9.3, 2.0, 2.1, 2.2.

Gem::InstallError: moab-versioning requires Ruby version ~> 2.1.

@cbeer
Copy link
Contributor Author

cbeer commented Feb 11, 2016

Do we know who the Ruby 1.9 and 2.0 consumers are? I assumed we were addressing them with the grand argo roll-out?

@jmartin-sul
Copy link
Contributor

looking at the projects listed in the dor-services dependencies ticket, it seems at first glance like the following support ruby 1.9:
https://github.com/sul-dlss/assembly
https://github.com/sul-dlss/robot-suite-base
https://github.com/sul-dlss/dor-services-app
https://github.com/sul-dlss/hydrus/blob/master/.ruby-version

from what @atz was saying the other day, it seems like hydrus won't make the transition to dor-services 5. and it sounded in the meeting this morning like the plan was to start using the current ruby version for dor-services-app. what about the other two? is the intention to upgrade those? i'd be in favor of doing that and dropping ruby 1.9 support in dor-services, if feasible.

@cbeer
Copy link
Contributor Author

cbeer commented Feb 12, 2016

Assembly is running under travis exclusively with ruby 2.1. I'll get a new version of ruby available on the robots VMs (and try to determine if there are any we can clean up...)

atz added a commit that referenced this pull request Feb 12, 2016
Update to moab-versioning 2.x, using fully namespaced Moab classes and modules
@atz atz merged commit 0ea7d88 into develop Feb 12, 2016
@atz atz deleted the moab-versioning-2x branch February 12, 2016 02:35
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.

4 participants