Skip to content
This repository has been archived by the owner on Jun 27, 2019. It is now read-only.

use default update interval (daily) #210

Merged
merged 1 commit into from
Dec 19, 2017
Merged

Conversation

xavdid
Copy link
Contributor

@xavdid xavdid commented Dec 18, 2017

See this comment. We don't want to bug users every single time they run a CLI command.

@xavdid xavdid requested a review from eliangcs December 18, 2017 20:44
Copy link
Member

@eliangcs eliangcs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't pull as this is a simple and straightforward change. The CI failed, but we can ignore it. Rerunning it a couple more times would make it pass. We should address the problem (CI build failed) in another PR.

@xavdid
Copy link
Contributor Author

xavdid commented Dec 19, 2017

Yeah, I'm not sure why the build command is taking so long recently. Did we change something there?

@eliangcs
Copy link
Member

@xavdid not that I'm aware of. We didn't change the build command recently.

@xavdid xavdid merged commit 72c8639 into master Dec 19, 2017
@xavdid xavdid deleted the update-notifier-less-often branch December 19, 2017 17:47
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.

2 participants