Add documentation and minor refactoring to allow PDFR to pass `devtools::check()` without errors about pdfr HOT 2 OPEN

The package is still kicking back a note and two warnings:

• Note: "Found if() conditions comparing class() to string"
• Significant warning: "utilities.cpp:186:26: warning: bitwise or with non-zero value always evaluates to true [-Wtautological-bitwise-compare]"
• Additional warning: "Non-staged installation was used"

I started addressing the note by replacing class(pdf) == "character" with rlang::is_character() with the assumption that an additional dependency for rlang should be fine if you are already importing ggplot2. I'd also be glad to convert the stop() errors into abort() or (if you're OK with adding a dependency on cli) cli::cli_abort(). Again, happy to handle these in a separate pull request or avoid adding these additional dependencies altogether if you prefer.

from pdfr.

Hi Eli

Thanks very much for this. I had rather given up developing this package as you can probably tell. I will have to re familiarise myself with the code base. Please do open a pull request; I'm very happy to have your input

Allan

from pdfr.