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

Pass an item to PublicDescMetadataService.new #472

Merged
merged 2 commits into from
Jan 9, 2019
Merged

Pass an item to PublicDescMetadataService.new #472

merged 2 commits into from
Jan 9, 2019

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Jan 9, 2019

Previously we were passing an instance of PublishMetadataService.
This lead to a NoMethodError.
Fixes #471

@jcoyne
Pass an item to PublicDescMetadataService.new
6e25bb2 
Previously we were passing an instance of PublishMetadataService.
This lead to a NoMethodError.
Fixes #471
@coveralls
Copy link

coveralls commented Jan 9, 2019 •
edited
Loading

Coverage Status

Coverage remained the same at 91.779% when pulling 0a86cf7 on pass_object into c47d72e on master.

@mjgiarlo mjgiarlo merged commit d5990b1 into master Jan 9, 2019
@mjgiarlo mjgiarlo deleted the pass_object branch January 9, 2019 21:03
@jcoyne jcoyne mentioned this pull request Jan 11, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mjgiarlo mjgiarlo mjgiarlo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jcoyne @coveralls @mjgiarlo