Update sw.js - Githubissues

publiclab / plots2

a collaborative knowledge-exchange platform in Rails; we welcome first-time contributors! :balloon:

https://publiclab.org

GNU General Public License v3.0

961 stars 1.83k forks source link

Update sw.js #11477

Closed yukiyuli closed 2 years ago

yukiyuli commented 2 years ago

replacing the var keywords with let/const

Fixes #11473

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

[ ] PR is descriptively titled 📑 and links the original issue above 🔗
[ ] tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
[ ] code is in uniquely-named feature branch and has no merge conflicts 📁
[ ] screenshots/GIFs are attached 📎 in case of UI updation
[ ] ask @publiclab/reviewers for help, in a comment below

welcome[bot] commented 2 years ago

Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help. Dangerbot will test out your code and reply in a bit with some pointers and requests. Also please refer here for installation help 💿 There may be some errors, but don't worry! We'll work through them with you! 👍🎉😄
One thing that can help to get started is to make sure you've included a link back to the original issue you're solving, in the format fixes #0000 (for example). And to make sure the PR title describes what you're trying to do! (often it can be the same as the issue title) Thanks! 🙌
Then, you can say hello in our chatroom & share a link to this PR to get a review! 👋 ✅

gitpod-io[bot] commented 2 years ago

codeclimate[bot] commented 2 years ago

Code Climate has analyzed commit 7fc86ffa and detected 0 issues on this pull request.

View more on Code Climate.

TildaDares commented 2 years ago

Hi @yukiyuli, thank you for taking the time to work on this PR but the issue has been resolved in #11476. If you'd like to work on another issue, please leave a comment in #11105 and we'll look for another one for you.