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

Upgrade to dor-services 7 #294

Merged
merged 1 commit into from
May 21, 2019
Merged

Upgrade to dor-services 7 #294

merged 1 commit into from
May 21, 2019

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Apr 18, 2019

Fixes #314

@jcoyne jcoyne force-pushed the test-dor-services-7 branch 4 times, most recently from 4f36955 to dc0fdca Compare April 22, 2019 15:00
@jcoyne jcoyne changed the title Test on dor-services 7 Upgrade to dor-services 7 May 15, 2019
@jcoyne jcoyne marked this pull request as ready for review May 15, 2019 13:47
@coveralls
Copy link

coveralls commented May 15, 2019

Coverage Status

Coverage decreased (-2.6%) to 91.286% when pulling efb0bec on test-dor-services-7 into f9f791f on master.

@jcoyne jcoyne force-pushed the test-dor-services-7 branch 6 times, most recently from 5dfd259 to abc886d Compare May 20, 2019 13:40
Copy link
Contributor

@justinlittman justinlittman left a comment

Choose a reason for hiding this comment

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

Approve pending consideration of comments and fixing conflicts.

@@ -104,7 +104,7 @@ namespace :hydrus do
task :refreshfix do
ts = [
'hydrus:loadfix',
'hydrus:refresh_workflows',
# 'hydrus:refresh_workflows',
Copy link
Contributor

Choose a reason for hiding this comment

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

Cruft?

puts resp.inspect
end
end
# desc 'refresh workflow datastreams'
Copy link
Contributor

Choose a reason for hiding this comment

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

Cruft?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, it's been excised in #320

@jcoyne jcoyne force-pushed the test-dor-services-7 branch 3 times, most recently from e47b48c to 240385a Compare May 21, 2019 17:29
@jcoyne jcoyne mentioned this pull request May 21, 2019
1 task
@justinlittman justinlittman merged commit 92ff292 into master May 21, 2019
@justinlittman justinlittman deleted the test-dor-services-7 branch May 21, 2019 19:33
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.

Upgrade hydrus to dor-services 7
3 participants