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.

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

fix: remove update-notifier #1706

Merged
merged 1 commit into from
Jun 30, 2022
Merged

fix: remove update-notifier #1706

merged 1 commit into from
Jun 30, 2022

Commits on Jun 30, 2022

  1. fix: remove update-notifier 

    it's better to catch outdated dependencies before deployment, with tools such as dependabot, instead of only finding about it when you're about to run the application

the motivation for having update-notifier isn't strong as per probot#1102 (comment) and it pulls in needless dependencies as showcased in probot#1703, therefore it's better to remove it
    @joao-paulo-parity
    joao-paulo-parity committed Jun 30, 2022
    Configuration menu
    Copy the full SHA
    2d53d64 View commit details
    Browse the repository at this point in the history