This PR contains some small updates from my review of the package and its tests:
Add dependencies to the DESCRIPTION following @blairj09’s parallel branch.
Moved some functions out of utils.R. I moved functions that were used in one part of the code, especially if they seemed to be a core part of that functionality.
The functions to generate messages went into openapi.R
parseQS() and combine_keys() went into reroute.R.
I updated some error messages in tableau_handler.R.
Added a test to test-tableau_json.R to complete its coverage.
This PR contains some small updates from my review of the package and its tests:
DESCRIPTION
following @blairj09’s parallel branch.utils.R
. I moved functions that were used in one part of the code, especially if they seemed to be a core part of that functionality.openapi.R
parseQS()
andcombine_keys()
went intoreroute.R
.tableau_handler.R
.test-tableau_json.R
to complete its coverage.