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.

Sign up for GitHub

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

Jump to bottom

Avoid notifying about updates on every run #120

Merged

akalipetis merged 1 commit into main from updates-less-noisy

Sep 26, 2023

Contributor

pjcdawkins commented Sep 22, 2023

Do not notify about updates on every run: notify only after checking for a new release (every hour by default).
Remove hardcoded state path ($HOME/.platformsh/state.json) and set it based on config.
Respect YAML config for updates.check and updates.check_interval if set in the embedded or environment config (however, this does not check the user config file).


          Avoid notifying about updates on every run

38cf08f

Also applies config to where the state is stored instead of a hardcoded path

akalipetis approved these changes

View reviewed changes

Member

akalipetis left a comment

LGTM 🐻

akalipetis merged commit 5834a86 into main

akalipetis deleted the updates-less-noisy branch

September 26, 2023 14:46

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

Labels

None yet