@cored nice. As a general rule of thumb what I would suggest is having methods be command or query methods, *in relation to themselves*. So that means unless the point of the method was to query the flay state I would have this method return `self`, as in the `Devtools::Tasks::Flay` instance.

I also try to minimize query methods to just those needed by collaborators. If I can I try to change them into command methods; like if I have a case where I query the state of the object and conditionally perform the object's method, I would want to change that to something where I just tell it to do the work and have it decide whether it's a noop or not.