Closed deepakunni3 closed 4 years ago
Looks good, thanks Deepak!
I think we'll need to remove this line from the Jenkinsfile too
I think we'll need to remove this line from the Jenkinsfile too
took the liberty, hope that's okay
@justaddcoffee Thanks for merging this in 👍
Okay.. good catch on removing the extra pigz
line.
But we should be removing both the pigz lines since the merge step will generate merged-kg.tar.gz
and merged-kg.nt.gz
by itself.
Okay, no problem - made a quick PR here: https://github.com/Knowledge-Graph-Hub/kg-covid-19/pull/311
This, along side latest changes on deepakunni3/kgx repo should get our pipeline going without the long merge step.