-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
Preloaded store #90
Preloaded store #90
Conversation
|
||
createSave(pathFS)(fetch$) | ||
.toPromise() | ||
.then(() => process.stdout.write('The end.\n')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Par curiosité, pourquoi process.stdout.write plutôt que console.log ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pour feinter le linter je dirais ^^.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
console.log c'est utilisé pour debugger. Là, je fais un CLI et c'est avec STDOUT que je veux travailler. J'ai assez subtil et discutable.
src/test/fallback.js
Outdated
@@ -22,3 +23,16 @@ test("shouldn't throw error if file doesn't exist", async t => { | |||
.toPromise(); | |||
t.deepEqual(events, []); | |||
}); | |||
|
|||
test("should load preloadedStore if fallback doesn't exists", async t => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exists --> exist
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rajoute peut-être un test dans le cas où preloadedStore est undefined/null ?
42317ac
to
6d577c3
Compare
Write ETCD folder in `preloadedStore` format. | ||
|
||
```Shell | ||
$ squirrel-dump --hosts localhost:2379 /etcd-folder ./dump.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Et du coup cette commande va aller dans un cron en prod ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
elle va aller dans la phase de build du mooc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok
No description provided.