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.

Sign up for GitHub

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

Jump to bottom

remove rpart_train() #1231

Merged

topepo merged 1 commit into main from no-rpart-train

Jan 20, 2025

+5 −141

Member

topepo commented Jan 19, 2025

From #1044 (already closed)


          changes for #1044

221506b

topepo commented

View reviewed changes

tests/testthat/test-decision_tree.R

-              test_that('rpart_train is stop-deprecated when it ought to be (#1044)', {
-                skip_on_cran()
-                # once this test fails, transition `rpart_train()` to `deprecate_stop()`

Member Author

topepo Jan 19, 2025

The trigger

topepo marked this pull request as ready for review

January 20, 2025 22:47

topepo merged commit 082351b into main

12 checks passed

topepo deleted the no-rpart-train branch

January 20, 2025 22:47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet