Fix heif converter being broken #55

Merged
retoor merged 1 commits from BordedDev/snek:bugfix/fix-pictures-again into main 2025-06-10 05:40:53 +02:00
Contributor

Register HEIF opener (guessing it got removed because it looked like an unused import) and handle terminal import error gracefully on windows

Register HEIF opener (guessing it got removed because it looked like an unused import) and handle terminal import error gracefully on windows
BordedDev added 1 commit 2025-06-09 22:04:37 +02:00
retoor reviewed 2025-06-10 05:39:38 +02:00
@ -4,3 +4,3 @@
import asyncio
import subprocess
import pty
try:
Owner

If i wouldn't use pty, but subprocess.pipe, would it work on windows while for Linux it ain't an option? I mean, does my docker setup work on windows?

If i wouldn't use pty, but subprocess.pipe, would it work on windows while for Linux it ain't an option? I mean, does my docker setup work on windows?
Author
Contributor

It's a good question, but I assume the docker stuff works if pty could be imported

It's a good question, but I assume the docker stuff works if pty could be imported
Owner

Does really not seem unused. Good job. Merged with my phone! 🥳🥳🥳

Does really not seem unused. Good job. Merged with my phone! 🥳🥳🥳
retoor merged commit 2fc18801a7 into main 2025-06-10 05:40:53 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: retoor/snek#55
No description provided.