Skip to content

Modernizations#4

Merged
mrhheffernan merged 12 commits intomasterfrom
matt/modernize
Mar 8, 2026
Merged

Modernizations#4
mrhheffernan merged 12 commits intomasterfrom
matt/modernize

Conversation

@mrhheffernan
Copy link
Owner

@mrhheffernan mrhheffernan commented Mar 7, 2026

Modernizing this repo since it's the most popular code I've put together.

Primary focus is on modernizing the dependencies, but also just making things more usable by adding uv to the mix for dependency management and execution.

There was a ton of really hacky, poorly written code in the base version. I've gone through and refactored basically everything so it now is much more efficient and is more pythonic. I've also removed some of the incredibly poor design choices, like using os.system calls to run scripts instead of just parsing in memory.

This PR is basically a complete overhaul of the code to bring it 8 years into the future and from a hackathon project to something a little more polished.

@mrhheffernan mrhheffernan marked this pull request as ready for review March 8, 2026 07:14
@mrhheffernan mrhheffernan merged commit d36981b into master Mar 8, 2026
2 checks passed
@mrhheffernan mrhheffernan deleted the matt/modernize branch March 8, 2026 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant