IT-VBFK

Results 59 comments of IT-VBFK

Think about coverage and qodana scan, if you can :)

Hmm.. IMHO `DateTimeOffset dateTime = 10.December(2011).Offset(0).At(09, 30, 45, 123, 456, 700);` doesn't read that fluent anymore. What about `10.December(2011)` returns `DateOnly`, `10.December(2011).At(09, 30, 45, 123, 456, 700)` `DateTime` and `10.December(2011).At(09,...

> Furthermore, DateOnly is not available in older versions of .NET, so that we would have to add a package for older versions. @jnyrup Can we polyfill this?

> The problem with returning DateOnly is that is does not implicitly casts to DateTime, so DateTime dt = 11.June(2017); will not compile anymore. Ok, what about adding the `WithOffset(x)`...

I think we should probably wait with this until v7 is out, because now (for v6) if a string mismatches the XUnit failure message is much more useful :) But...

I really like how option 2 looks like, because it's so explicit about lifetime and such 👍

@stijnwop Did you gave up on this? This would be a great addition.

For me, deleting all previous stuff fixes this issue: e.g. `rm -rf token_extractor_docker*` This "orphaned" files remains on disk, e.g. if token extractor quitted with an unexpected error..