Fix issue #672 : Refactor "filer/tests/spec/fs.mkdir.spec.js" to follow ES6 syntax with "const/let" instead of "var" #722
Conversation
Codecov Report
@@ Coverage Diff @@
## master #722 +/- ##
=======================================
Coverage 86.71% 86.71%
=======================================
Files 16 16
Lines 1746 1746
=======================================
Hits 1514 1514
Misses 232 232Continue to review full report at Codecov.
|
humphd
left a comment
There was a problem hiding this comment.
Nice work, this is great.
| const fs = util.fs(); | ||
| expect(fs.promises.mkdir('/tmp')).to.be.a('promise'); | ||
| }); | ||
| }); |
There was a problem hiding this comment.
One tip: it's a good idea to always include a newline at the end of all your files (I know this isn't your mistake, it was already like this). We do this, and GitHub shows its absence, because we want to avoid having concatenated files break when the end of one file and the start of another get joined. Just a tip for next time.
There was a problem hiding this comment.
Sounds good, will do.
strict modeletandconstinstead ofvar