adding jsonlite as a dep

cpsievert / pitchRx

Tools for scraping MLB Gameday data and Visualizing PITCHf/x

http://cpsievert.github.io/pitchRx/

Other

123 stars 33 forks source link

adding jsonlite as a dep #71

Closed james-ingold closed 4 years ago

james-ingold commented 4 years ago

I'm definitely not the most experienced R developer and I think I forgot to include jsonlite as a dependency on the last pull request. I was having an issue pulling down a clone copy of the master branch to a clean box. @cpsievert can you take a look and verify I'm importing jsonlite correctly?

cpsievert commented 4 years ago

I use roxygen to maintain the NAMESPACE and man/ files, so please add the import somewhere around here instead of in the NAMESPACE directly

https://github.com/cpsievert/pitchRx/blob/master/R/scrape.R#L28

james-ingold commented 4 years ago

Thanks for taking a look. I updated the pr