import syntax and ckanserviceprovider version #125
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi everyone, it's me again!
So I was trying to build and run the container for datapusher-plus and it failed in two ways:
config
module frommain
. Indeed:config
is not a globally registered module and therefore cannot be imported asconfig
unless$CWD
isdatapusher
, which I suspect most of the time is not. This PR fixes that. The fix assumes that this is installed as a python package, probably viapip install -e
or similar, which is what the readme says anyways. I'm actually wondering how this worked to begin with.ckanserviceprovider
1.2.0 (see ckan/ckan-service-provider@b467d44).Thanks!