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

Remove jsontableschema dependency #58

Merged
merged 10 commits into from Aug 9, 2016
Merged

Remove jsontableschema dependency #58

merged 10 commits into from Aug 9, 2016

Conversation

roll
Copy link
Member

@roll roll commented Aug 9, 2016

It fixes #51

Instead of having processors.Schema(schema) now we have processors.Convert(converter) to convert string to python objects. It could be used also with JTS:

schema = jsontableschema.Schema(descriptor)
topen('path.csv', processors=[processors.Convert(schema.convert)])

@roll
Copy link
Member Author

roll commented Aug 9, 2016

@georgiana-b
could you please review

@coveralls
Copy link

Coverage Status

Coverage increased (+2.2%) to 91.215% when pulling af5d1fe on remove-jts-dep into 65746c1 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+2.2%) to 91.215% when pulling af5d1fe on remove-jts-dep into 65746c1 on master.

@coveralls
Copy link

coveralls commented Aug 9, 2016

Coverage Status

Coverage increased (+2.2%) to 91.215% when pulling e596983 on remove-jts-dep into 65746c1 on master.

@coveralls
Copy link

coveralls commented Aug 9, 2016

Coverage Status

Coverage increased (+2.2%) to 91.215% when pulling 2278647 on remove-jts-dep into 65746c1 on master.

@coveralls
Copy link

coveralls commented Aug 9, 2016

Coverage Status

Coverage increased (+2.2%) to 91.152% when pulling 1470a1d on remove-jts-dep into e15c331 on master.

@coveralls
Copy link

coveralls commented Aug 9, 2016

Coverage Status

Coverage increased (+2.2%) to 91.152% when pulling b475280 on remove-jts-dep into e15c331 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+2.2%) to 91.152% when pulling b475280 on remove-jts-dep into e15c331 on master.

@coveralls
Copy link

coveralls commented Aug 9, 2016

Coverage Status

Coverage increased (+2.2%) to 91.152% when pulling b475280 on remove-jts-dep into e15c331 on master.

@roll roll merged commit 552b38b into master Aug 9, 2016
@roll roll removed the progress label Aug 9, 2016
@roll roll deleted the remove-jts-dep branch August 9, 2016 16:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove jsontableschema dependency?
3 participants