Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upUse readtext for other extensions #28
Conversation
The idea is to use `readtext` whenever it explicitly supports a format (list taken from their manual)
This comment has been minimized.
This comment has been minimized.
I think this accomplishes the task described in the heading, so I think we should be fine merging. However, I did test this with the following as a test file:
and it returns an empty list, which means we end up with an
Which should not happen anyway (if we have a 'NULL' it should quietly ignore the entry). I suggest we merge this and add an issue either to include the empty link boilerplate (which really should go away anyway) or use a partial function (if R has that or a similar feature) to disinclude all null entries in the lapply statement. The problem with XML can be catalogued as part of #18 (part of the solution would be to try and improve the regex). (If you agree just thumbs up and I'll merge). |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Weird - I got a broken merge warning in my email, but checks passed according to the website. |
This comment has been minimized.
This comment has been minimized.
Problem fixed itself after reset. Seemed like a blip with the R package for Travis. |
adam3smith commentedOct 17, 2019
The idea is to use
readtext
whenever it explicitly supports a format (list taken from their manual)