Close file before removing to satisfy Windows requirements - Linked: #27

[engineerjoe440]

Description:

As illustrated in #27, this PR adds the os.close(...) to handle closing files before attempting to remove them to satisfy Windows requirements. From local testing, this solves the observed issue, quite nicely.

Changes:

• Add os.close(...) for both the temporary file and temporary SVG file.

Linked Issues:

• Closes PermissionError ... cannot access the file because it is being used by another process: on Windows #27

Closing Thoughts:

• I was unable to get pre-commit working on my system. Ended up having a number of failures that prevented the commit. So this PR might need an extra-close review to make sure the formatting requirements, etc., are met in the way that you want.
• THANK YOU, for this AWESOME project! It's wonderful!!!

[ewels]

Super nice, thanks! 🎉

[ewels]

I was unable to get pre-commit working on my system.

No worries - it runs with GitHub Actions anyway, so will automatically pick up and problems once you open the PR.

[ewels]

Thanks again for this! Just pushed a release v1.2.2 which contains the fix.