Comment 6 for bug 1926819

Revision history for this message
Aaron Thompson (dev-aaront-org) wrote :

Thanks for the response Lucas. I would prefer to see something to the effect that the system is up to date, because that at least tells me that update-notifier-common is there and working. But if the decision is to not write anything in that case, then I strongly agree with Daniel. The 3 blank lines are really noticeable and they make it look like something went wrong. It looks like something is supposed to be there but isn't. IMO there should be 1 or 0 blank lines. Anything more and it looks like a potential bug.

I would suggest that the tests from my patch (or similar) be added to the codebase, updated with the intended output of course. Such tests would make it clear that this change was intentional and not accidental.