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

Import Handling #471

Merged
merged 4 commits into from
May 7, 2019
Merged

Import Handling #471

merged 4 commits into from
May 7, 2019

Conversation

beckyjackson
Copy link
Contributor

@beckyjackson beckyjackson commented May 1, 2019

Fixes import loading for SPARQL UPDATEs and across chained commands. See #468

@beckyjackson
Copy link
Contributor Author

I ran the command causing the Travis failure on my system:

$ robot query --input nucleus.owl --update update.ru --output results/nucleus_update.owl
$ robot diff --left nucleus_update.owl --right results/nucleus_update.owl 
Ontologies are identical

When diffing, the ontologies are the same. I re-ran mvn verify and all tests passed.

@jamesaoverton if you get a chance, can you run mvn verify on your system and see if you get a different result?

Copy link
Member

@jamesaoverton jamesaoverton left a comment

Choose a reason for hiding this comment

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

This looks good. Just two small suggestions.

@jamesaoverton jamesaoverton merged commit 8e6f460 into ontodev:master May 7, 2019
@beckyjackson beckyjackson deleted the update-fix branch May 23, 2019 20:03
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.

None yet

2 participants