Fix datetime printing issue

[gowerc]

Closes #132

[gowerc]

I think from memory this code exists (as opposed to just using as.character() ) to force the timestamp to be shown. Regular dates already have the NA conversion covered within the default as.character() handler but for some reason that was missed here

[github-actions]

Code Coverage Summary

Filename                Stmts    Miss  Cover    Missing
--------------------  -------  ------  -------  --------------------------
R/ascii_tables.R          117       8  93.16%   10, 153, 164, 169-172, 240
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                     702      37  94.73%

Results for commit: f9f0c06

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

[github-actions]

Unit Tests Summary

  1 files   13 suites   7s ⏱️
 57 tests  53 ✅  4 💤 0 ❌
591 runs  580 ✅ 11 💤 0 ❌

Results for commit f9f0c06.

♻️ This comment has been updated with latest results.

[bundfussr]

Hi @gowerc , @kieranjmartin , when do you plan to merge and release the bug fix?

The bug prevents displaying the differences in the admiralroche CI/CD jobs.

[gowerc]

I think Kieran is back off holiday next week so this should be merged then, I was also hoping to resolve #135 & #133 and then release it as a bug patch, most of those seem pretty simple to fix so maybe end of next week for release (or start of week after if anything comes up).

[bundfussr]

I think Kieran is back off holiday next week so this should be merged then, I was also hoping to resolve #135 & #133 and then release it as a bug patch, most of those seem pretty simple to fix so maybe end of next week for release (or start of week after if anything comes up).

Sounds good. Thanks.

[kieranjmartin]

minor changes, just to expand the testing a little to ensure we have coverage

[gowerc]

@kieranjmartin - Should be good for re-review

[gowerc]

@bundfussr - This is now on CRAN

[bundfussr]

@bundfussr - This is now on CRAN

Great, thanks!