Skip to content

Conversation

@gowerc
Copy link
Owner

@gowerc gowerc commented Aug 26, 2025

Closes #133

@github-actions
Copy link
Contributor

github-actions bot commented Aug 26, 2025

badge

Code Coverage Summary

Filename                Stmts    Miss  Cover    Missing
--------------------  -------  ------  -------  -----------------
R/ascii_tables.R          117       4  96.58%   10, 153, 164, 240
R/cast_variables.R         49       0  100.00%
R/diffdf.R                197       2  98.98%   122, 407
R/generate_keyname.R       12       0  100.00%
R/identify.R              152       8  94.74%   283-290
R/is_different.R           52       0  100.00%
R/issuerows.R              40       0  100.00%
R/issues.R                 17       1  94.12%   52
R/misc_functions.R         34       2  94.12%   9, 13
R/print.R                  38       4  89.47%   54-55, 58-59
TOTAL                     708      21  97.03%

Results for commit: 9e8c634

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Contributor

github-actions bot commented Aug 26, 2025

Unit Tests Summary

  1 files   13 suites   7s ⏱️
 60 tests  56 ✅  4 💤 0 ❌
597 runs  585 ✅ 12 💤 0 ❌

Results for commit cd9a3df.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@kieranjmartin kieranjmartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion around default naming

@gowerc gowerc requested a review from kieranjmartin August 27, 2025 21:33
@gowerc gowerc merged commit 02dba73 into master Oct 15, 2025
@gowerc gowerc deleted the 133-fix-deparse branch October 15, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error from diffdf::diffdf related to deparse(substitute())

3 participants