Skip to content

Conversation

AlexTMjugador
Copy link
Member

@AlexTMjugador AlexTMjugador commented Oct 5, 2025

Overview

We currently maintain our own fork of wry with three significant changes:

Since all these changes can now be implemented using standard APIs that don't require forking, and our wry fork is no longer being used in the latest release version (someone updated our Cargo.lock to use a newer upstream version, causing a build warning about the unused patch), this PR implements these changes using the discussed standard APIs, eliminating the maintenance burden of keeping our fork up to date, and preventing such patching failures from going unnoticed in the future.

@AlexTMjugador AlexTMjugador requested review from a team and Gaming32 October 5, 2025 13:37
@AlexTMjugador AlexTMjugador added app Relates to Modrinth App DevEx Improvements to developer experience labels Oct 5, 2025
@AlexTMjugador AlexTMjugador marked this pull request as ready for review October 5, 2025 14:04
@AlexTMjugador AlexTMjugador added this pull request to the merge queue Oct 5, 2025
Merged via the queue into main with commit dbc64af Oct 5, 2025
4 checks passed
@AlexTMjugador AlexTMjugador deleted the alex/wry-patch-fork-removal branch October 5, 2025 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app Relates to Modrinth App DevEx Improvements to developer experience

Development

Successfully merging this pull request may close these issues.

2 participants