Skip to content

stores the check_update_interval as a .env variable#31

Open
AngeloBonini wants to merge 1 commit into
rodbecc:mainfrom
AngeloBonini:self-update-env
Open

stores the check_update_interval as a .env variable#31
AngeloBonini wants to merge 1 commit into
rodbecc:mainfrom
AngeloBonini:self-update-env

Conversation

@AngeloBonini
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Collaborator

@vvasc vvasc left a comment

Choose a reason for hiding this comment

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

Looks good. I think you should add some lines in readme to explain how to use it

Comment thread tests/self_update.test.ts
@@ -0,0 +1,17 @@
import { spy } from 'mock';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

you can remove it and add the tests in a new PR

Comment thread .env.defaults
@@ -0,0 +1 @@
CHECK_UPDATE_INTERVAL= No newline at end of file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

do we need the default one?

Comment thread .gitignore
@@ -1 +1,2 @@
last_check_update.txt No newline at end of file
last_check_update.txt
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

shouldn't be better if we use the same .env file do get this too?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants