search

DomT4 / homebrew-autoupdate

:tropical_drink: An easy, convenient way to automatically update Homebrew.
BSD 2-Clause "Simplified" License
995 stars 55 forks source link

Logging in addition to notifications #15

Closed jdejager closed 3 years ago

jdejager commented 6 years ago

Wouldn't it be useful to add some logging? I've been using autoupdate for a couple of weeks now, but it leaves me wondering what was updated...

I've activated the notifications, but I never see them since the updates usually run when I'm not at my computer.

jdejager commented 6 years ago

FYI ... Besides this inconvenience, I love it! Should have been part of brew from the get go IMO.

DomT4 commented 6 years ago

Sorry for the delay here; I watch far too many massively active projects on GitHub & then wonder why I constantly have to dig my way through several thousand notifications.

There should be logs present in ~/Library/Logs/com.github.domt4.homebrew-autoupdate?

SimonBrazell commented 4 years ago

Would it be possible to have a quick summary in the notification?

DomT4 commented 4 years ago

Would it be possible to have a quick summary in the notification?

I’m on holiday this week but I’ll look into it once I’m back. Might be able to get the notification to read the last line of the log or something, perhaps.

SimonBrazell commented 4 years ago

Yeah I thought that too so I had a quick look at the logs after I posted this but couldn't find a nice summary of the updates done for the days I looked at.

16 Jan 2020 8:54:34 pm Dominyk Tiller notifications@github.com:

Would it be possible to have a quick summary in the notification?

I’m on holiday this week but I’ll look into it once I’m back. Might be able to get the notification to read the last line of the log or something, perhaps.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub [https://github.com/DomT4/homebrew-autoupdate/issues/15?email_source=notifications&email_token=AD6N4L2SXJSANXYZXZT7FJDQ6AVFRA5CNFSM4EXKVCOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJDOXWA#issuecomment-575073240] , or unsubscribe [https://github.com/notifications/unsubscribe-auth/AD6N4LYHWYOGBSAATTULPPTQ6AVFRANCNFSM4EXKVCOA] . [https://github.com/notifications/beacon/AD6N4L5ICDCZDZ2STLLYTWLQ6AVFRA5CNFSM4EXKVCOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJDOXWA.gif]

SimonBrazell commented 4 years ago

Perhaps we could store the output of brew outdated before the update is done and then use this in the notification? Great work on this utility btw!

RafaelMiquelino commented 4 years ago

Sorry for the delay here; I watch far too many massively active projects on GitHub & then wonder why I constantly have to dig my way through several thousand notifications.

There should be logs present in ~/Library/Logs/com.github.domt4.homebrew-autoupdate?

@DomT4, I issued a PR to include this in the docs. Would it be possible to accept that?

DomT4 commented 4 years ago

I'll try and get back into the broader conversation on this handling of notifications in the near future, but I've accepted @RafaelMiquelino's PR to point the log location out more clearly in the meantime.

DomT4 commented 3 years ago

Perhaps we could store the output of brew outdated before the update is done and then use this in the notification? Great work on this utility btw!

It's not there yet but the new notification system I've started working on contains an update on which taps were updated. Fuller explanation in https://github.com/DomT4/homebrew-autoupdate/issues/25#issuecomment-729088027 and the referenced commits.

DomT4 commented 3 years ago

Folded into https://github.com/Homebrew/homebrew-autoupdate/issues/43.