Yeah, that would be preferrable and something I tried to accomplish originally (see other TimeUnit enums in backend repo). The problem is that we might not want to support exectly all those enums in all related classes or functions. In that sense, I think it's good to have context-specific enums.

I'm in for either path though, so I'll leave it up to you to choose :) Just keep in mind the next time we add some function that needs a WEEK, or MILLIS unit, we're in for quite a bit of additional work ;)