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

Handle DenialConstraints in MetacrateResultReceiver #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

f4lco
Copy link

@f4lco f4lco commented Nov 25, 2017

Originally I was looking into metanome-cli which has a dependency on de.hpi.isg:mdms-metanome. I was unable to build this module against a recent version of Metanome due to an extension of the OmniscientResultReceiver interface. This is my suggested fix.

EDIT: I've just noticed through Travis this concerns not yet released functionality (Metanome master; seemingly 1.1-SNAPSHOT is not deployed regularly?), so this PR is not too relevant I guess? Feel free to close.

@sekruse
Copy link
Contributor

sekruse commented Nov 26, 2017

Thanks for the PR. I guess, once the current Metanome master is deployed in the Sonatype snapshot repository, the build should pass. I will merge this PR then.

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.

2 participants