As mentioned in #2485, I guess instead of hard-coding 24 hours(1 day), we could store a config key inside the gh config file. That would conform to the issue better (IMHO).

@shrihanDev gh already stores the last time a network check has been perfomed in state.yml.
This PR makes it read that timestamp to avoid noticing a new release if said check was performed less than 24hs ago.

@cristiand391 Sorry for not being clear at first, but actually I meant that instead of 24 hours, we could let the user decide the interval. The user could set a config key that stores the time interval, and then gh should read the value of that config key and do apropos.

Sorry if I miss something again 😢

@shrihanDev no need to apologize, sorry if I sounded rude 😅.
I went with 24hs because that's how frequently gh checks for update.

I can't see how a user would benefit from customizing it, but if more users ask for it then maybe the cli team will consider adding it.

@cristiand391 No probs! Let's wait for more votes to this customisation.

The one logical customization I can see is being able to disable the update checks in the config file, not just via env vars. I agree w/ @cristiand391 there doesn't seem to be much value in fine tuning the frequency of update checks as a user.

Otherwise I was about to put up a PR doing the same thing as this one based on @mislav's helpful pointer in #2485, when I saw this one was already here :)

So stupid of me! I was thinking that this PR adds the functionality to check for release builds in each repository, and notify once a day 🤦🏻‍♂️

Extremely sorry.

Now that I know what's going on, I guess it's pointless to customise the frequency of the new-release-checking.

Thank you for your contribution!

I've pushed a commit to simplify things. I wanted reduce the responsibilities of getLatestReleaseInfo and move the StateFile handling into CheckForUpdate. This made it easier to read and write the file only when necessary and IMO makes the logic easier to follow and reason about. It is now easy to see how if the StateFile shows that we checked for an update less than 24 hours ago we will short circuit the rest of the CheckForUpdate function.