search

skalnik / aqi-wtf

😷 WTF is the AQI near me right now?!
https://aqi.wtf
MIT License
44 stars 12 forks source link

Remove unused var and comments for clarity #58

Closed agillette closed 3 years ago

agillette commented 3 years ago

I noticed an unused variable and wanted to add a couple of notes where things were confusing for me.

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/skalnik/aqi-wtf/mzkuja5sw
✅ Preview: https://aqi-wtf-git-fork-agillette-clean-up.skalnik.vercel.app

obra commented 3 years ago

Thank you!

The code cleanup is +1.

The comment pointing at the equation: I feel like there's got to be a better, more authoritative source than a forum post. This is absolutely better than nothing, though.