Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include --start interval and other options in --status command #38

Open
NathanUrwin opened this issue Jan 20, 2021 · 3 comments
Open

Include --start interval and other options in --status command #38

NathanUrwin opened this issue Jan 20, 2021 · 3 comments

Comments

@NathanUrwin
Copy link

Title says it all. Thanks for this awesome software!

@DomT4
Copy link
Owner

DomT4 commented Feb 1, 2021

I'd need to store the options invoked initially somewhere and then read those options back, but I agree this would be a handy feature. I've been meaning to give users a way of seeing when they invoked the command as well, as sometimes changes/etc require a fresh command to roll out. An updated print out like:

~> brew autoupdate --status                                                                              [~]
Autoupdate is installed and running.

Autoupdate is scheduled to update every 43200 seconds.
Autoupdate was last invoked on 01/01/20 with the options:
  --start --enable-notifications

Would probably be alright. I'll look into it. Thanks for the issue, and I'm glad you're finding this project useful! 😺

DomT4 added a commit that referenced this issue Feb 26, 2021
@swissbuechi
Copy link
Contributor

I'd need to store the options invoked initially somewhere and then read those options back, but I agree this would be a handy feature. I've been meaning to give users a way of seeing when they invoked the command as well, as sometimes changes/etc require a fresh command to roll out. An updated print out like:

~> brew autoupdate --status                                                                              [~]
Autoupdate is installed and running.

Autoupdate is scheduled to update every 43200 seconds.
Autoupdate was last invoked on 01/01/20 with the options:
  --start --enable-notifications

Would probably be alright. I'll look into it. Thanks for the issue, and I'm glad you're finding this project useful! 😺

You are already storing all the information you need in the .plist and the homebrew_autoupdate files.
I will try to parse the content from these files in the status class, so we can to output the requested information when the command is invoked.

@swissbuechi
Copy link
Contributor

I've already implemented --immediate and interval on my fork: https://github.com/swissbuechi/homebrew-autoupdate/tree/enhance-output-of-status-command

image

I will try to finish all other options and then create a pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants