Remove TealData and check argument

[llrs-roche]

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.

[github-actions]

✅ All contributors have signed the CLA
_{Posted by the CLA Assistant Lite bot.}

[llrs-roche]

I have read the CLA Document and I hereby sign the CLA

[github-actions]

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  -------------------
R/cdisc_data.R                   1       0  100.00%
R/deprecated.R                  14       0  100.00%
R/dummy_function.R               2       2  0.00%    14-15
R/formatters_var_labels.R       61       0  100.00%
R/join_key.R                    38       0  100.00%
R/join_keys-c.R                 12       0  100.00%
R/join_keys-extract.R          128       0  100.00%
R/join_keys-names.R             15       0  100.00%
R/join_keys-parents.R           30       0  100.00%
R/join_keys-print.R             45       0  100.00%
R/join_keys-utils.R             73       3  95.89%   35-38
R/join_keys.R                   21       0  100.00%
R/teal_data-class.R             35       4  88.57%   71, 119-121
R/teal_data-constructor.R       11       2  81.82%   31, 35
R/teal_data-extract.R            3       0  100.00%
R/teal_data-get_code.R          13       8  38.46%   114-120, 124
R/teal_data-names.R              8       1  87.50%   31
R/teal_data-show.R               4       4  0.00%    14-19
R/testhat-helpers.R             26       0  100.00%
R/topological_sort.R            32       0  100.00%
R/verify.R                      42      11  73.81%   67, 97-101, 104-108
TOTAL                          614      35  94.30%

Diff against main

Filename                     Stmts    Miss  Cover
-------------------------  -------  ------  -------
R/deprecated.R                 -57     -57  +80.28%
R/teal_data-constructor.R      -20     -14  +33.43%
TOTAL                          -77     -71  +9.64%

Results for commit: d7780ee

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[github-actions]

Unit Tests Summary

  1 files   15 suites   1s ⏱️
163 tests 163 ✅ 0 💤 0 ❌
218 runs  218 ✅ 0 💤 0 ❌

Results for commit d7780ee.

♻️ This comment has been updated with latest results.

[m7pr]

Thanks @llrs-roche I will review on Monday

[m7pr]

Hey @llrs-roche so many functions were removed. Can you update NEWS.md with a note about it?

[m7pr]

@llrs-roche some things got merge to the main branch, now you have conflicts with the main branch. Would you be able to resolve conflicts locally and push?

[m7pr]

Hey, left a couple of comments

[llrs-roche]

I've updated the PR addressing all the comments. I included a brief point in NEWS.md too.

[github-actions]

Unit Test Performance Difference

Test Suite	$Status$	Time on main	$±Time$	$±Tests$	$±Skipped$	$±Failures$	$±Errors$
deprecated	💀	$0.10$	$-0.10$	$-2$	$0$	$0$	$0$

Additional test case details

Test Suite	$Status$	Time on main	$±Time$	Test Case
deprecated	💀	$0.01$	$-0.01$	getter_returns_same_as_names_
deprecated	💀	$0.08$	$-0.08$	setter_does_nothing_to_object

Results for commit 20d8e21

♻️ This comment has been updated with latest results.