Hello, Marcin Borkowski <mb...@mbork.pl> writes:
> I decided to put a warning about this in the docstring in my patch. My > assumption was that this is enough. If a user wants to change the > default, he will most probably see the docstring and will have to > actively ignore the warning. I don't think this is enough. As you put it, it instills doubt without explaining anything. Why 35? Why "may not work"? What "may not work"? > I do not use the vanilla Org way of selecting tasks to clock from > history - I use org-mru-clock, for which the restriction doesn't make > sense. So we have something else to use, I meant "have something else to use out of the box". I.e., not with an external library. I agree with the idea of removing this limitation, but I'm not convinced your patch is ideal yet. Regards, -- Nicolas Goaziou