Skip to content

fix: path discovery issues - #5

Merged
aoqia194 merged 4 commits into
LeafPZ:mainfrom
Stikulzon:main
Oct 14, 2026
Merged

aoqia194 merged 4 commits into
LeafPZ:mainfrom
Stikulzon:main

Conversation

@Stikulzon

Copy link
Copy Markdown
Contributor

Fixed startup problems when the proxy JAR is located away from the game JAR, like in .leaf/lib/, a broken discovery of the workshop mod. Proxy now sets location of gameJarPath for loader as well.

@aoqia194 aoqia194 changed the title Fix path discovery issues fix: path discovery issues Oct 11, 2026
@aoqia194 aoqia194 added this to Global Oct 11, 2026
@aoqia194 aoqia194 moved this from triage to in progress in Global Oct 11, 2026
@aoqia194 aoqia194 added this to the v0.2.2 milestone Oct 11, 2026
@aoqia194 aoqia194 added priority/high Should not be ignored risk/medium Might break something but not in a major way status/needs-review Is reay for maintainer review type/fix labels Oct 11, 2026
@aoqia194 aoqia194 self-assigned this Oct 11, 2026
@Stikulzon

Copy link
Copy Markdown
Contributor Author

Needs testing on Linux

@aoqia194 aoqia194 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are good changes, but some things here I've questioned

@aoqia194 aoqia194 added status/changes-requested A reviewer found changes that must be done and removed status/needs-review Is reay for maintainer review labels Oct 14, 2026
@aoqia194

Copy link
Copy Markdown
Member

Also I've tested on Linux and it is working.

@Stikulzon

Copy link
Copy Markdown
Contributor Author

Should be good now

@aoqia194 aoqia194 removed the status/changes-requested A reviewer found changes that must be done label Oct 14, 2026
@aoqia194
aoqia194 self-requested a review October 14, 2026 13:18

@aoqia194 aoqia194 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for the changes.

@aoqia194 aoqia194 moved this from in progress to review in Global Oct 14, 2026
@aoqia194
aoqia194 merged commit 31b177c into LeafPZ:main Oct 14, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from review to done in Global Oct 14, 2026
@aoqia194 aoqia194 modified the milestones: v0.2.2, v0.3.0 Oct 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority/high Should not be ignored risk/medium Might break something but not in a major way type/fix

Projects

Status: done

Development

Successfully merging this pull request may close these issues.

2 participants