Rick M

Oh okay, now I see what you mean

I'd suggest calling this icon path validator or icon name validator, since it's not actually loading/rendering the icon(s).

You don't seem to have checked out your repo.

Completely changing the location of methods is API breaking. Users have to re-code things to make them work again. While it is minimal effort to adapt it's still a breaking...

I'll have to pull the branch and play with it. If, as you've said, the methods are still accessed the same way then I totally agree it's non-breaking. I took...

🤦‍♂️ I'm being brain dead. The unit tests would have been different if there were API changes. I know they did move but the faker usage is the same. So...

I haven't looked but is this PR branched after that fix or does it need rebasing?

datafaker-net/datafaker#1180 was merged "3 Days ago" (that's all the web UI shows). So hard to know.

I agree that getting rid of `java.sql.Timestamp` usage is likely a good move. Is there a reason to move away from `java.util.Date`? (Not that I'm against it, I just want...

Thanks, great resources. Makes sense.