Skip to content
This repository has been archived by the owner on Aug 28, 2021. It is now read-only.

Implement generic json import #20

Closed
aboodman opened this issue Jul 9, 2015 · 2 comments
Closed

Implement generic json import #20

aboodman opened this issue Jul 9, 2015 · 2 comments
Assignees

Comments

@aboodman
Copy link
Contributor

aboodman commented Jul 9, 2015

Lots of data out there is JSON. You should be able to dump it into noms and work with it.

Seems like we'd have to import it as just loosely typed lists and maps. Leaving this as milestone:later since it's not very clear what form this should take right now.

See also #19 and #21.

@aboodman aboodman added this to the later milestone Jul 9, 2015
This was referenced Jul 9, 2015
@cmasone-attic cmasone-attic self-assigned this Jul 9, 2015
@cmasone-attic
Copy link
Contributor

Seems like this will drive some useful demos, so taking a pass at it

cmasone-attic added a commit that referenced this issue Jul 9, 2015
This basic importer takes a URL and pulls the JSON in as
weakly typed data -- lists and maps of basic types.

Addresses issue #20
cmasone-attic added a commit that referenced this issue Jul 9, 2015
1) Tolerate lists with no entries
2) Exit with error messages instead of Chk.NoError() in main()
3) General variable names
4) Use json.NewDecoder() instead of reading all data into memory again.
5) Idiomatic fixes.

Addresses issue #20
@aboodman
Copy link
Contributor Author

aboodman commented Jul 9, 2015

woo

@aboodman aboodman closed this as completed Jul 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants