Get rid of star imports.

These are dangerous as you will import not only the functions defined
in the other module but also all of it's imports! This makes automated
cleanups brittle.

bin/validity-led-dance

@@ -5,7 +5,9 @@ import os
 from binascii import unhexlify
 from enum import Enum
 from validitysensor.flash import read_flash
-from validitysensor.sensor import *
+from validitysensor.sensor import glow_start_scan, glow_end_scan
+from validitysensor.util import assert_status
+from validitysensor.tls import tls
 from validitysensor import init
 from time import sleep
 from usb import core as usb_core

validitysensor/tls.py

@@ -3,7 +3,7 @@ import os
 import pickle
 import logging
 from struct import pack, unpack
-from binascii import *
+from binascii import hexlify, unhexlify
 
 from cryptography.hazmat.primitives.ciphers import Cipher, algorithms, modes
 from cryptography.hazmat.backends import default_backend

validitysensor/usb.py

@@ -1,7 +1,7 @@
 import logging
 import errno
 import usb.core as ucore
-from binascii import *
+from binascii import hexlify, unhexlify
 from .util import assert_status
 from struct import unpack
 from usb.core import USBError