<html><head></head><body><div style="color:#000; background-color:#fff; font-family:Helvetica Neue, Helvetica, Arial, Lucida Grande, sans-serif;font-size:13px"><div>Hello Gary and everyone,</div><div id="yui_3_16_0_ym19_1_1534609098985_5755"><br></div><div dir="ltr" id="yui_3_16_0_ym19_1_1534609098985_5758">I've been looking through the commit log and I keep seeing PEP8. Then I was reading through some merge requests that were not implemented; more PEP8. Gary, you mentioned in a MR comment that there were a lot of PEP8 errors in the pinup script. I looked up PEP8 and see that it is a style guide for Python.</div><div dir="ltr" id="yui_3_16_0_ym19_1_1534609098985_5759"><br></div><div dir="ltr" id="yui_3_16_0_ym19_1_1534609098985_5733">Since I don't know PEP8 really at all, I found a online compliance checker. It is <a href="http://pep8online.com/" class="" id="yui_3_16_0_ym19_1_1534609098985_5732">http://pep8online.com/</a>. Clockmaker is littered with (~80) errors. Is using an online checker recommend ? Is this one as good as any other? I could imagine that some are more accurate than others. <br></div><div dir="ltr" id="yui_3_16_0_ym19_1_1534609098985_5869"><br></div><div dir="ltr" id="yui_3_16_0_ym19_1_1534609098985_5868">I thought I would work on the clockmaker script regarding PEP8. That would be a first step to integrating the pinup functionality into clockmaker. It doesn't make much sense to me to have 2 scripts, with one being a superset of the other. My $dayjob has taught me that it is guaranteed such a situation will lead to non-conformance/out of sync issues. In fact, I just submitted a MR because the Pi 3B+ H/W is in pinup but not in clockmaker. <br></div><div id="yui_3_16_0_ym19_1_1534609098985_5880"><br></div><div id="yui_3_16_0_ym19_1_1534609098985_7104">Enjoy your weekend. <br></div><div id="yui_3_16_0_ym19_1_1534609098985_7113"><br></div><div id="yui_3_16_0_ym19_1_1534609098985_6003">Regards,</div><div dir="ltr" id="yui_3_16_0_ym19_1_1534609098985_6052">Mike<br></div></div></body></html>