Ryan Blue
Ryan Blue
Looks good. I'll merge when tests are passing.
Thanks, @findepi!
I like this a lot, but I think we should make some adjustments so that the output is usable with other CLI tools. It feels weird to work with a...
Okay, I think I've reviewed all of the PRs. Lots of stuff going in!
Looks great! I know we're still waiting on config, but this is usable right now so I merged it.
@stevenzwu and @openinx, can you review this?
I don't agree with this change. I think it's good to know that this is skipped, rather than not being able to distinguish between missing logs and not trying to...
I would like a lot to distinguish between "didn't try" and "nothing to clean up" -- even when "nothing to clean up" is because there were no expired files.
Thanks, @ajantha-bhat!
@namrathamyske, it looks like some of these changes went in with #5364. Can you rebase on that?