Remove TealData and check argument (#349)

# Pull Request Fixes #344 The first commit is to deal with TealData and connectors: code and documentation removed. The second commit is to deal with `check` argument: I've kept the documentation of `verify()` all the other related code and documentation has been removed. All these functions already had `deprecate_stop` so they would have caused an error on calling them and they have been so for ~1year so I think directly removing them is fine. Local check (with teal.code 0.5.0.9013 ) doesn't raise warnings, errors or notes. --------- Signed-off-by: Lluís Revilla <[email protected]> Co-authored-by: Marcin <[email protected]>
insightsengineering · Nov 11, 2024 · ce21695 · ce21695
1 parent f026675
commit ce21695
Show file tree

Hide file tree

Showing 10 changed files with 17 additions and 633 deletions.
diff --git a/NAMESPACE b/NAMESPACE
@@ -23,70 +23,18 @@ S3method(parents,teal_data)
 S3method(print,join_keys)
 export("col_labels<-")
 export("datanames<-")
-export("get_join_keys<-")
 export("join_keys<-")
 export("parents<-")
-export(as_cdisc)
-export(callable_code)
-export(callable_function)
 export(cdisc_data)
-export(cdisc_data_connector)
-export(cdisc_data_file)
-export(cdisc_dataset)
-export(cdisc_dataset_connector)
-export(cdisc_dataset_connector_file)
-export(cdisc_dataset_file)
-export(code_cdisc_dataset_connector)
-export(code_dataset_connector)
 export(col_labels)
 export(col_relabel)
-export(csv_cdisc_dataset_connector)
-export(csv_dataset_connector)
-export(data_connection)
 export(datanames)
-export(dataset)
-export(dataset_connector)
-export(dataset_connector_file)
-export(dataset_file)
 export(example_cdisc_data)
-export(fun_cdisc_dataset_connector)
-export(fun_dataset_connector)
-export(get_attrs)
-export(get_cdisc_keys)
-export(get_dataname)
-export(get_dataset)
-export(get_dataset_label)
-export(get_datasets)
-export(get_join_keys)
-export(get_key_duplicates)
-export(get_keys)
-export(get_labels)
-export(get_raw_data)
-export(is_pulled)
 export(join_key)
 export(join_keys)
-export(load_dataset)
-export(load_datasets)
-export(mae_dataset)
-export(mutate_data)
-export(mutate_dataset)
 export(parent)
 export(parents)
-export(python_cdisc_dataset_connector)
-export(python_code)
-export(python_dataset_connector)
-export(rds_cdisc_dataset_connector)
-export(rds_dataset_connector)
-export(read_script)
-export(relational_data_connector)
-export(script_cdisc_dataset_connector)
-export(script_dataset_connector)
-export(set_args)
-export(set_keys)
 export(teal_data)
-export(teal_data_file)
-export(to_relational_data)
-export(validate_metadata)
 export(verify)
 exportMethods(get_code)
 exportMethods(show)

diff --git a/NEWS.md b/NEWS.md
@@ -5,6 +5,7 @@
 - Soft deprecate `datanames` argument of `get_code()`. Use `names` instead.
 - Soft deprecate of `datanames()`. Use `names()` instead.
 - Deprecate of `datanames(x) <- value`. Does nothing, replace with renaming the objects inside the environment.
+- All parameters and functions deprecated on 0.4.0 were removed.
 
 
 ### Enhancements

diff --git a/R/cdisc_data.R b/R/cdisc_data.R
@@ -32,7 +32,6 @@
 #'
 cdisc_data <- function(...,
                        join_keys = teal.data::default_cdisc_join_keys[names(rlang::list2(...))],
-                       code = character(0),
-                       check) {
-  teal_data(..., join_keys = join_keys, code = code, check = check)
+                       code = character(0)) {
+  teal_data(..., join_keys = join_keys, code = code)
 }