Remove type casting to factor when few values (numeric)

[Melkiades]

What changes are proposed in this pull request?

• Removed type casting to factor when few values (numeric).

#79

---

Pre-review Checklist (if item does not apply, mark is as complete)

• All GitHub Action workflows pass with a ✅
• PR branch has pulled the most recent updates from master branch: usethis::pr_merge_main()
• If a bug was fixed, a unit test was added.
• Code coverage is suitable for any new functions/features (generally, 100% coverage for new code): devtools::test_coverage()
• Request a reviewer

Reviewer Checklist (if item does not apply, mark is as complete)

• If a bug was fixed, a unit test was added.
• Run pkgdown::build_site(). Check the R console for errors, and review the rendered website.
• Code coverage is suitable for any new functions/features: devtools::test_coverage()

When the branch is ready to be merged:

• Update NEWS.md with the changes from this pull request under the heading "# cards (development version)". If there is an issue associated with the pull request, reference it in parentheses at the end update (see NEWS.md for examples).
• All GitHub Action workflows pass with a ✅
• Approve Pull Request
• Merge the PR. Please use "Squash and merge" or "Rebase and merge".

[github-actions]

Unit Tests Summary

  1 files   66 suites   1m 34s ⏱️
 66 tests  66 ✅ 0 💤 0 ❌
163 runs  163 ✅ 0 💤 0 ❌

Results for commit 050de08.

♻️ This comment has been updated with latest results.

[github-actions]

Unit Test Performance Difference

Additional test case details

Test Suite	$Status$	Time on main	$±Time$	Test Case
tbl_hierarchical_rate_by_grade	💚	$7.65$	$-2.58$	tbl_hierarchical_rate_by_grade_works
theme_gtsummary_roche	👶		$+0.00$	with_assign_summary_type_arg_cat_threshold_0L_each_data_type_is_always_mapped_as_it_is

Results for commit 93c66ba

♻️ This comment has been updated with latest results.

[github-actions]

Code Coverage Summary

Filename                               Stmts    Miss  Cover    Missing
-----------------------------------  -------  ------  -------  ---------
R/add_blank_rows.R                        63       0  100.00%
R/add_hierarchical_count_row.R            33       0  100.00%
R/crane-package.R                          2       2  0.00%    17-18
R/deprecated.R                             6       6  0.00%    15-21
R/label_roche.R                           86       4  95.35%   174-177
R/modify_header_rm_md.R                   18       2  88.89%   35-36
R/modify_zero_recode.R                    13       0  100.00%
R/tbl_baseline_chg.R                     184       0  100.00%
R/tbl_hierarchical_rate_and_count.R      148       0  100.00%
R/tbl_hierarchical_rate_by_grade.R       271       3  98.89%   162-164
R/tbl_listing.R                           35       0  100.00%
R/tbl_null_report.R                        9       0  100.00%
R/tbl_roche_summary.R                     60       0  100.00%
R/tbl_shift.R                            115       0  100.00%
R/tbl_survfit_quantiles.R                132       1  99.24%   295
R/tbl_survfit_times.R                     92       0  100.00%
R/theme_gtsummary_roche.R                 74       0  100.00%
R/utils.R                                  5       0  100.00%
TOTAL                                   1346      18  98.66%

Diff against main

Filename                     Stmts    Miss  Cover
-------------------------  -------  ------  --------
R/theme_gtsummary_roche.R       +1       0  +100.00%
TOTAL                           +1       0  +0.00%

Results for commit: 050de08

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[Melkiades]

forgot! apologies

[ddsjoberg]

Looks great! Thank you!