diffdf

diffdf copied to clipboard

update print message

6

I think on balance including "All rows are shown in table below" is unecessary, and we should only point out where this isn't the case

kieranjmartin

Add option to set number of observations printed per issue

2

gowerc

Convert backend to use data.table

7

One of the main changes we added in the now abandoned v2 branch was the use of `data.table`. If memory serves me right this resulted in a ~30% speed up...

gowerc

Use `data.table` backend

4

Closes #117

gowerc

Abstract report design

8

@kieranjmartin In order to support the generation of html / md as well as allowing users to customise the output as they see fit I think we need to abstract...

gowerc

Lets rid of double exclamation marks

3

I just think they're a bit much...

kieranjmartin

Add datetime unit tests

2

Closes #121 Main change is adding the timezone to the print out of datetime variables to make it explicit what they are showing as otherwise it leads to some confusing...

gowerc

Resolve R CMD check NOTES

2

Closes #122

gowerc

Add tests for niche datetime scenarios

2

I don't think we have any explicit handling / tests for datetime variables in particular for when they have different timezones. Whilst I suspect diffdf should behave as expected it...

gowerc

``` * checking R code for possible problems ... NOTE as_fmt_char.list : : no visible global function definition for ‘capture.output’ Undefined global functions or variables: capture.output Consider adding importFrom("utils", "capture.output")...

gowerc