Skip to content

Conversation

@dibenzepin
Copy link
Contributor

(I should probably split the formatting commit into another PR but i added the thing that does the formatting here...)

this closes #62 by fixing up the source build instructions (changing to Python 3, adding ImageMagick—I didn't bother with which because it's most likely already there), and adding a flake.nix to make reproduction easier.

And then I just added some formatters in the flake.nix (because why not), and ran them...they're Ruff, Mdformat, yamlfmt, and Nixfmt.

@samuelngs samuelngs self-assigned this Mar 15, 2025
@samuelngs
Copy link
Owner

@poopsicles LGTM. I just noticed the repo’s gitHub actions is outdated and broken, so the test failed. Gimme a day or two to pull the changes and double check everything, and I’ll merge it soon after!

Many thanks for the PR btw!

@samuelngs samuelngs merged commit 3230b2e into samuelngs:ios-17.4 Mar 17, 2025
1 check failed
@dmlls dmlls mentioned this pull request Apr 2, 2025
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.

Incomplete "from source" instructions in README

2 participants