Zach H comments

Results 246 comments of


                                            Zach H

[Feature Request] In chat, don't tag cards that have no match in the database.

Hmm, this would be very helpful. I might try and look into this

mtgjson doesn't respect Scryfall's rate limiting guidelines

This is an interesting revelation, as I don't recall seeing rate limits within the logs. Do you have a copy of any of your runs that can show the rate...

feat(scryfall): warn once we reach API's rate limit

Can you update your PR to pass the builds? Thanks!

Cockatrice should auto-load the last built deck

This is a nice to have, but I don't think we can dedicate active resources to this. If you (or any one else) would like to submit a PR to...

Migrate project management to `uv`

Hey there! I have a few questions regarding your changes. While I'm generally not a fan of changing our build pipelines as a first commit from a new author, I'm...

Scryfall rate limit is 10 req/s

Pending on #1333 to see if this is a problem with our build chain

Restoring connection reverses hand

If this is so (I haven't had time to deal with it yet) is it that big of a deal? Just curious

Restoring connection reverses hand

Hand sorting isn't kept on the server, it's a local thing. We don't want to send extra connection data.

[Fix-Warnings] Use auto instead of type

I'm indifferent on this, it seems fine

[Fix-Warnings] Local variable can be made const

Conflicts + fails to build. Also not certain this is fully necessary, but I won't stop it