Heavy review of package documentation and some minor implementation changes:
add license information (copied / adapted from dplyr)
clean up NAMESPACE issues
remove a bunch of old functions
add docs
add unit tests for the package
add many yaml tests for databases
change default_X_path to pkg_test and pkg_config for easier reference
fix a few arg names and arg order to make things more natural
alphabetized long lists to make finding imports / dependencies easier
this gets us very close to 0.1.0. Still a handful of things to wrestle through, but wanted to get your feedback on progress thus far!
You will notice I began breaking out the tests I am writing into multiple files. I think this follows testthat best practices and will make managing the tests easier in the long term. One note worth considering - the tests get very verbose with all of the verbs represented. However, I do not think there is a nice way to run the same logic through multiple verbs without getting really complicated, so I agree with your decision thus far to keep the verbosity in the interest of clarity.
Heavy review of package documentation and some minor implementation changes:
default_X_pathtopkg_testandpkg_configfor easier referencethis gets us very close to 0.1.0. Still a handful of things to wrestle through, but wanted to get your feedback on progress thus far!
You will notice I began breaking out the tests I am writing into multiple files. I think this follows
testthatbest practices and will make managing the tests easier in the long term. One note worth considering - the tests get very verbose with all of the verbs represented. However, I do not think there is a nice way to run the same logic through multiple verbs without getting really complicated, so I agree with your decision thus far to keep the verbosity in the interest of clarity.