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

don't use rdf-iiif gem: IIIF vocab part of rdf-vocab since 0.8.2 #194

Merged
merged 1 commit into from
May 13, 2015

Conversation

ndushay
Copy link
Contributor

@ndushay ndushay commented May 13, 2015

connected to #164
connected to sul-dlss-deprecated/rdf-iiif#1

@darrenleeweber

@dazza-codes
Copy link
Contributor

@ndushay - looks OK to me and glad to see it now in rdf/vocab - merge when satisfied the travis builds are OK.

WRT process, I'm working with http://datasift.github.io/gitflow/, so I prefer to see a feature branch merged into a develop branch rather than the master branch (reserve the master branch for working code that has been released).

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) to 99.68% when pulling 74ceffd on no-iiif-gem into 05ba2bd on master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) to 99.68% when pulling 74ceffd on no-iiif-gem into 05ba2bd on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) to 99.68% when pulling 74ceffd on no-iiif-gem into 05ba2bd on master.

ndushay added a commit that referenced this pull request May 13, 2015
don't use rdf-iiif gem:  IIIF vocab part of rdf-vocab since 0.8.2
@ndushay ndushay merged commit 6e2054c into master May 13, 2015
@ndushay ndushay deleted the no-iiif-gem branch May 13, 2015 20:07
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.

3 participants