Closed geeksville closed 4 years ago
easier to view here: https://github.com/ThingPulse/esp8266-oled-ssd1306/pull/291/files. If you would prefer me to squash into one commit I can do that.
If you would prefer me to squash into one commit I can do that.
No need to, thanks. We usually do a "Squash and merge" when we merge PRs.
Your video and my own very brief visual inspection is all I have to assess this. I don't have the hardware to test, but hey, LGTM.
(I just updated this old change based on current master and put it in a separate feature branch)
ESP32 targets are still supported (my project targets both)
Backstory here: https://github.com/ThingPulse/esp8266-oled-ssd1306/pull/277