Update instances of var to let/const

dwyl / learn-tape

:white_check_mark: Learn how to use Tape for JavaScript/Node.js Test Driven Development (TDD) - Ten-Minute Testing Tutorial

187 stars 31 forks source link

Update instances of var to let/const #27

Closed victoria-kaihe closed 5 years ago

victoria-kaihe commented 6 years ago

README and code could be updated to be in line with ES6+ use of let/const instead of var.

iteles commented 6 years ago

@victoria-kaihe Thanks for opening this issue! 🎉 This is a great suggestion.

If you have some time to put in a PR to update this, it would be very welcome!

victoria-kaihe commented 6 years ago

I've cloned down the repo and checked out to a new branch to do the update. I'm having a quick look through now to get an estimate at how long it'll take. Once I've got a timeframe I'll open up a PR and get stuck in!

victoria-kaihe commented 6 years ago

I'm thinking a couple of hours, maybe three? Opening up a PR now :)

nelsonic commented 5 years ago

Resolved. Closing. Thanks! ✨