Skip to content

Conversation

@gowerc
Copy link
Owner

@gowerc gowerc commented Aug 26, 2025

Hey @kieranjmartin - not sure what email you wanted to use for this now so feel free to push a commit with your preferred choice.

In terms of the change to the license file itself I just ran usethis::use_mit_license() and stuck to what it gave me as default.

@gowerc gowerc requested a review from kieranjmartin August 26, 2025 20:50
@gowerc gowerc changed the title Update licence and contact emails Update license and contact emails Aug 26, 2025
@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          115       8  93.04%   10, 154, 165, 170-173, 239
R/cast_variables.R         49       0  100.00%
R/diffdf.R                209      18  91.39%   373-390, 417
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                  20       0  100.00%
TOTAL                     700      37  94.71%

Results for commit: 38eddfe

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   6s ⏱️
 55 tests  53 ✅ 2 💤 0 ❌
589 runs  580 ✅ 9 💤 0 ❌

Results for commit 38eddfe.

♻️ This comment has been updated with latest results.

@gowerc gowerc merged commit ab426e9 into master Aug 27, 2025
24 checks passed
@gowerc gowerc deleted the xxx-licence branch August 27, 2025 14:50
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.

3 participants