-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
consider relocating tests for tune_args()
and tunable()
#194
Comments
I think they are fine here because you only need workflows Imports/Suggests to test them and they are specific to workflows itself |
I think we needed the tests to be in extratests during the transition period of the various methods (or at least it was easier). I meant more "moving tests into workflows" rather than the other direction - if there's anything in those tests in extratests that isn't covered here already. |
* add postprocessors to tunable * units tests for tunable postproc and some work on #194 * required for calibration * Update tests/testthat/test-generics.R Co-authored-by: Simon P. Couch <[email protected]> --------- Co-authored-by: Simon P. Couch <[email protected]>
Originally posted by @hfrick in #193 (review)
The text was updated successfully, but these errors were encountered: