Skip to content

Conversation

jajik
Copy link
Member

@jajik jajik commented Aug 5, 2024

Opening as a draft, because there might be more.

@jajik jajik force-pushed the fix-locking branch 3 times, most recently from 4ed1b63 to aed3158 Compare August 8, 2024 06:40
@rhusar rhusar requested a review from jfclere August 12, 2024 07:08
@rhusar
Copy link
Member

rhusar commented Aug 13, 2024

@jajik Unrelated - but can you look into replacing strings such as "text/plain" with macros as well? I see you are changing lines that include it and it's repeated many times.. Thx.

@jajik
Copy link
Member Author

jajik commented Aug 14, 2024

@jajik Unrelated - but can you look into replacing strings such as "text/plain" with macros as well? I see you are changing lines that include it and it's repeated many times.. Thx.

Proposed in #266

@jajik jajik removed the request for review from jfclere October 2, 2024 14:35
@jajik jajik force-pushed the fix-locking branch 2 times, most recently from a0061f3 to caf1303 Compare October 4, 2024 13:44
@jajik jajik self-assigned this Mar 24, 2025
@jajik
Copy link
Member Author

jajik commented Oct 20, 2025

This needs a revisit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants