Run pylint and black

Author: mcauser

examples/basic_bitbang.py

@@ -17,28 +17,28 @@
 sr = SR74HC595_BITBANG(ser, srclk, rclk)
 
 sr.clear()  # raises RuntimeError because you haven't provide srclr pin
-sr.enable() # raises RuntimeError because you haven't provide oe pin
+sr.enable()  # raises RuntimeError because you haven't provide oe pin
 
 # reconstruct with all pins
-oe = Pin(33, Pin.OUT, value=0)    # low enables output
-srclr = Pin(32, Pin.OUT, value=1) # pulsing low clears data
+oe = Pin(33, Pin.OUT, value=0)  # low enables output
+srclr = Pin(32, Pin.OUT, value=1)  # pulsing low clears data
 
 sr = SR74HC595_BITBANG(ser, srclk, rclk, srclr, oe)
 
 sr.bit(1)  # send high bit, do not latch yet
 sr.bit(0)  # send low bit, do not latch yet
-sr.latch() # latch outputs, outputs=0000_0010
+sr.latch()  # latch outputs, outputs=0000_0010
 
-sr.bit(1, 1) # send high bit and latch, outputs=0000_0101
-sr.bit(0, 1) # send low bit and latch, outputs=0000_1010
+sr.bit(1, 1)  # send high bit and latch, outputs=0000_0101
+sr.bit(0, 1)  # send low bit and latch, outputs=0000_1010
 
-sr.bits(0xff, 4) # send 4 lowest bits of 0xff (sends 0x0f), outputs=1010_1111
+sr.bits(0xFF, 4)  # send 4 lowest bits of 0xff (sends 0x0f), outputs=1010_1111
 
-sr.clear(0) # clear the memory but don't latch yet
+sr.clear(0)  # clear the memory but don't latch yet
 sr.latch()  # next latch shows the outputs have been reset
 
-sr.bits(0b1010_1010, 8) # write some bits
+sr.bits(0b1010_1010, 8)  # write some bits
 sr.clear()  # clear the memory and latch, outputs have been reset
 
 sr.enable()  # outputs enabled
-sr.enable(0) # outputs disabled
+sr.enable(0)  # outputs disabled

examples/basic_spi.py

@@ -12,20 +12,20 @@
 spi = SPI(1, 100000)
 rclk = Pin(5, Pin.OUT)
 
-oe = Pin(33, Pin.OUT, value=0)    # low enables output
-srclr = Pin(32, Pin.OUT, value=1) # pulsing low clears data
+oe = Pin(33, Pin.OUT, value=0)  # low enables output
+srclr = Pin(32, Pin.OUT, value=1)  # pulsing low clears data
 
-sr = SR74HC595_SPI(spi, rclk, 2) # chain of 2 shift registers
+sr = SR74HC595_SPI(spi, rclk, 2)  # chain of 2 shift registers
 
-sr.pin(2,1)  # set pin 2 high of furthest shift register
-sr.pin(2)    # read pin 2
-sr.pin(2,0)  # set pin 2 low
+sr.pin(2, 1)  # set pin 2 high of furthest shift register
+sr.pin(2)  # read pin 2
+sr.pin(2, 0)  # set pin 2 low
 
-sr.toggle(8) # toggle first pin of closest shift register
+sr.toggle(8)  # toggle first pin of closest shift register
 
-sr[0] = 0xff # set all pins high on furthest shift register
+sr[0] = 0xFF  # set all pins high on furthest shift register
 sr[1] = 240  # set half pins high on closest shift register
-sr[1]        # read pins
+print(sr[1])  # read pins
 
 oe.value(0)  # disable outputs
 oe.value(1)  # enable outputs

examples/package.json

@@ -4,5 +4,5 @@
     ["sr74hc595/examples/basic_spi.py", "github:mcauser/micropython-74hc595/examples/basic_spi.py"]
   ],
   "deps": [],
-  "version": "1.0.0"
+  "version": "1.0.1"
 }

package.json

@@ -5,5 +5,5 @@
     ["sr74hc595/sr74hc595_spi.py", "github:mcauser/micropython-74hc595/src/sr74hc595_spi.py"]
   ],
   "deps": [],
-  "version": "1.0.0"
+  "version": "1.0.1"
 }

readme.md

@@ -80,7 +80,7 @@ If you don't need to clear the outputs, connect `srclr` to vcc.
 If you don't need to disable the outputs, connect `oe` to gnd.
 
 ```python
-__init__(spi, rclk, len=1, srclr=None, oe=None)
+__init__(spi, rclk, length=1, srclr=None, oe=None)
 ```
 
 Read the boolean value of a pin. First pin is index `0`. If you are cascading shift

src/sr74hc595_bitbang.py

@@ -6,15 +6,16 @@
 https://github.com/mcauser/micropython-74hc595
 """
 
-__version__ = '1.0.0'
+__version__ = "1.0.1"
+
 
 class SR74HC595_BITBANG:
     def __init__(self, ser, srclk, rclk, srclr=None, oe=None):
         self.ser = ser
         self.srclk = srclk
         self.rclk = rclk
         self.srclr = srclr  # tie high if functionality not needed
-        self.oe = oe        # tie low if functionality not needed
+        self.oe = oe  # tie low if functionality not needed
 
         self.ser.init(ser.OUT, value=0)
         self.srclk.init(srclk.OUT, value=0)
@@ -47,13 +48,13 @@ def latch(self):
 
     def clear(self, latch=True):
         if self.srclr is None:
-            raise RuntimeError('srclr pin is required')
+            raise RuntimeError("srclr pin is required")
         self.srclr(0)
         self.srclr(1)
         if latch:
             self.latch()
 
     def enable(self, enabled=True):
         if self.oe is None:
-            raise RuntimeError('oe pin is required')
+            raise RuntimeError("oe pin is required")
         self.oe(not enabled)

src/sr74hc595_spi.py

@@ -6,16 +6,17 @@
 https://github.com/mcauser/micropython-74hc595
 """
 
-__version__ = '1.0.0'
+__version__ = "1.0.1"
+
 
 class SR74HC595_SPI:
-    def __init__(self, spi, rclk, len=1, srclr=None, oe=None):
+    def __init__(self, spi, rclk, length=1, srclr=None, oe=None):
         self.spi = spi
         self.rclk = rclk
         self.srclr = srclr  # tie high if functionality not needed
-        self.oe = oe        # tie low if functionality not needed
+        self.oe = oe  # tie low if functionality not needed
 
-        self.buf = bytearray(len)
+        self.buf = bytearray(length)
         self.rclk.init(rclk.OUT, value=0)
 
         if self.srclr is not None:
@@ -36,26 +37,26 @@ def pin(self, pin, value=None, latch=True):
         if value is None:
             return (self.buf[pin // 8] >> (pin % 8)) & 1
         elif value:
-            self.buf[pin // 8] |= (1 << (pin % 8))
+            self.buf[pin // 8] |= 1 << (pin % 8)
         else:
             self.buf[pin // 8] &= ~(1 << (pin % 8))
         self._write(latch)
 
     def toggle(self, pin, latch=True):
-        self.buf[pin // 8] ^= (1 << (pin % 8))
+        self.buf[pin // 8] ^= 1 << (pin % 8)
         self._write(latch)
 
     def clear(self, latch=True):
         if self.srclr is None:
-            raise RuntimeError('srclr pin is required')
+            raise RuntimeError("srclr pin is required")
         self.srclr(0)
         self.srclr(1)
         if latch:
             self.latch()
 
     def enable(self, enabled=True):
         if self.oe is None:
-            raise RuntimeError('oe pin is required')
+            raise RuntimeError("oe pin is required")
         self.oe(not enabled)
 
     def __getitem__(self, index):