search

adafruit / Adafruit_CircuitPython_AzureIoT

Access to Microsoft Azure IoT device, messaging, and job services from CircuitPython!
MIT License
20 stars 15 forks source link

Fixed pylint failures - mostly line-too-long #54

Closed evaherrada closed 2 years ago

evaherrada commented 2 years ago

Def needs to be looked over after it is merged to make sure all the documentation still looks ok.

github-actions[bot] commented 2 years ago

👋 Thanks for this pull request! Unfortunately, it looks like the automated continuous integration (CI) test(s) failed. These can be tricky to fix so we've written a guide on how to fix them locally. It has pages about running pre-commit locally and another about building the docs locally with sphinx. Thanks for contributing to CircuitPython! If you have more questions, feel free to join the Adafruit Discord and post in #circuitpython-dev.

evaherrada commented 2 years ago

oops forgot to do the examples

evaherrada commented 2 years ago

Not entirely sure about this failure. Someone else may be able to fix it much quicker than me

tekktrik commented 2 years ago

Looks like mpy-cross is failing to compile the library, I'll look into this one

tekktrik commented 2 years ago

The issue it's having with compiling the .mpy files seems to be related to have f-strings on multiple lines. I can make some suggestions after looking more into it, but honestly it'll be worth considering disabling the "line too long" warning for those lines and leaving a comment about why that was done.