Skip to content

Conversation

@gowerc
Copy link
Owner

@gowerc gowerc commented Aug 26, 2025

Closes #135

@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                193       1  99.48%   400
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                     704      20  97.16%

Results for commit: 1a4659e

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 ⏱️
 59 tests  55 ✅  4 💤 0 ❌
595 runs  584 ✅ 11 💤 0 ❌

Results for commit 1a4659e.

♻️ 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.

Minor coding change suggested in testing

@gowerc gowerc merged commit 19a4dab into master Aug 27, 2025
23 checks passed
@gowerc gowerc deleted the 135-move-file-arg branch August 27, 2025 19:16
@dwoollacott
Copy link

Thanks @gowerc & @kieranjmartin - do you have a planned release date for the change?

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.

only 10 rows of differences shown when file= argument is used

4 participants