Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

feat: flag and config to skip notifications #1024

Closed
wants to merge 3 commits into from
Closed

feat: flag and config to skip notifications #1024

wants to merge 3 commits into from

Conversation

jlucktay
Copy link

@jlucktay jlucktay commented Oct 7, 2022

At the end of a run, topgrade normally sends a system notification.
This change adds a command-line flag and a config-file option to disable this behaviour.

Fixes #936.

Standards checklist:

  • The PR title is descriptive.
  • The code compiles (cargo build)
  • The code passes rustfmt (cargo fmt)
  • The code passes clippy (cargo clippy)
  • The code passes tests (cargo test)
  • Optional: I have tested the code myself
    • I also tested that Topgrade skips the step where needed

If you developed a feature or a bug fix for someone else and you do not have the
means to test it, please tag this person here.

At the end of a run, topgrade normally sends a system notification.
This change adds a command-line flag and a config-file option to
disable this behaviour.
@jlucktay
Copy link
Author

jlucktay commented Oct 9, 2022

Hi there @r-darwish! Is there anything else I can do to help get this merged/released?

@jlucktay jlucktay closed this by deleting the head repository Oct 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide option to disable system notification on Mac
2 participants