Don't load lubridate on startup + other upkeep #237
Merged
Add this suggestion to a batch that can be applied as a single commit. This suggestion is invalid because no changes were made to the code. Suggestions cannot be applied while the pull request is closed. Suggestions cannot be applied while viewing a subset of changes. Only one suggestion per line can be applied in a batch. Add this suggestion to a batch that can be applied as a single commit. Applying suggestions on deleted lines is not supported. You must change the existing code in this line in order to create a valid suggestion. Outdated suggestions cannot be applied. This suggestion has been applied or marked resolved. Suggestions cannot be applied from pending reviews. Suggestions cannot be applied on multi-line comments. Suggestions cannot be applied while the pull request is queued to merge. Suggestion cannot be applied right now. Please check back later.
This should fix #116 , supersedes #163 (I think it's better to rely on cli's inline theme, rather than RStudio, as it makes the package more portable.
Using how tidyverse loads its packages rather than base R to create less clutter when loading the package.
Removing packages from depends
You will note that I tweaked the vignettes loading scheme.
Showing library(tidyverse), library(tidyquant)
actually running library(dplyr) library(lubridate) etc. library(tidyquant)
I also added
skip()to avoid test failures when APIs are not available.Move
tq_performance()andtq_transmute()lists of functions into an internal data, as I assume that xts, quantmod are quite stable and if there are any new functions, it is always possible to re-run the data-raw script and update the possible functions.Addresses #235
Edit: closes #91