jawn icon indicating copy to clipboard operation
jawn copied to clipboard

Csv reader

Open benjaminettori opened this issue 9 years ago • 5 comments

Hi,

I took at stab at the csv to json reader. Not sure if this is what we are trying to do.

Also, I am having trouble integrating tape with travis-ci. The build fails and so does npm test. However if I run the tests locally they all pass. Any ideas?

benjaminettori avatar Mar 02 '16 00:03 benjaminettori

Hey Ben,

I think the issue might be that your JavaScript isn't standardized. I had the same problem as well.

Here's a link for how to format JavaScript:

http://standardjs.com/rules.html

mels065 avatar Mar 02 '16 07:03 mels065

Hi thanks for the link. I think it's weird that travis enforces no semicolons at the end of the line. I used to work with lint, and it enforced the exact opposite rule!

benjaminettori avatar Mar 02 '16 12:03 benjaminettori

Haha yeah I know! I'm so used to using semicolons.

mels065 avatar Mar 02 '16 15:03 mels065

Travis is running both standardjs and travis. Here are the details for your build: https://travis-ci.org/CfABrigadePhiladelphia/jawn/jobs/113008574

It's failing with some style complaints from standardjs.

flyingzumwalt avatar Mar 02 '16 16:03 flyingzumwalt

Oops. Didn't see your comments!

I bet we can change the settings for standardjs if we have our own style preferences, or we could just turn it off.

flyingzumwalt avatar Mar 02 '16 16:03 flyingzumwalt