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