Code Cleanup

[tcyrus]

Made a lot of changes while looking through the source code

• replaced random.randint with random.getrandbits where appropriate
• replace catch AttributeError with getattr where appropriate
• pyproject.toml: fixed mixed indentation of classifiers
• pyproject.toml: specified minimum setuptools version
• EmbThread: created properties to replace getter + setter functions
• EmbThread: fixed issue where color could be set to None
• EmbPattern: replaced os.path with pathlib.PurePath
• EmbPattern: fixed minor bug in add_stitch_absolute
• EmbPattern: use type narrowing for stream argument (read_embroidery and write_embroidery)
• EmbPattern: fixed bug where __iadd__ would sometimes return None
• test: removed from __future__ import print_function
• test: use tempfile with context manager instead of os.remove
• test: avoid wildcard imports where possible
• test: use unittest.skip for TestReadHus + test_generic_write_gcode

[tcyrus]

Fixed the issues and separated the commits. Passes all tests locally.

[tcyrus]

EmbThread.color can be set to None in some situations where color_hex can't parse a string. This causes errors in the EmbThread dunder methods since they depend on EmbThread.color being an integer. The EmbThread.color attribute has been replaced with a property, which means that the setter can stop this situation from happening. I've removed the fallback value (0x000000) for parse_string_color so that this behavior is consistent and not just for hex strings.

Also, io.TextIOWrapper can be used to read and write to a binary file-object as if it was a text file-object. This can be used to replace the READ_FILE_IN_TEXT_MODE and WRITE_FILE_IN_TEXT_MODE attributes.

[kaalleen]

Thank you so much for all the work you've put into this. This package really needs someone to take the time and do this kind of work on it.

It's all looking good to me.