FR: Support library("tidyverse") notation

[stephlocke]

Speech marks are not supported in the CRAN or git version (as of Issue generation point). Taking the example from parse_packages:

> cat('library("ggplot2")\n # library(curl)\n require(leaflet)\n CB::date_print()\n',file='temp.R')
> parse_packages('temp.R')
[1] "\"ggplot2\"" "leaflet"     "CB"         
> unlink('temp.R')

This should result in

> cat('library("ggplot2")\n # library(curl)\n require(leaflet)\n CB::date_print()\n',file='temp.R')
> parse_packages('temp.R')
[1] "ggplot2" "leaflet"     "CB"         
> unlink('temp.R')

[stephlocke]

If I have time to write tests etc this is how I'd change the library call (and obv the require would also need to be changed)

(?<=library\("*)[0-9|a-z|A-Z|.]+

Example: https://regexr.com/3s9ua

[cole-brokamp]

Thanks for bringing this up. I hope to have some time during the next week to implement this fix. Pull request always welcome if you want to take a stab before I get to it.

[stephlocke]

omg Cole - no tests present 😱

I'm gonna start your test harness as I have some time right now. Any objections to testthat?

[cole-brokamp]

Please do! I’ve never used it but always wanted to.

…

On Jul 12, 2018, at 11:51 AM, Stephanie Locke ***@***.***> wrote: omg Cole - no tests present 😱 I'm gonna start your test harness as I have some time right now. Any objections to testthat? — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

[stephlocke]

This is now working in #5, just waiting for travis to complete

[cole-brokamp]

thank you for showing me the testing ways @stephlocke 😎 merged!