Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

initial build fixes for BetterJunimosForestry #11

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jacob-keller
Copy link

This attempts to fix the BetterJunimosForestry mod to work with Stardew
Valley 1.6, though it is currently build-tested only.

The big changes include:

  • switching to string IDs instead of numerical indexes
  • fixing many function references
  • re-wrote the fruit tree harvest logic entirely
  • removed a bunch of references to debris logic that appears to have
    been removed from Stardew Valley 1.6
  • Update the target to net6.0, and update the ModBuildConfig and JSON
    references.
  • Various other fixes require to get things compiling

Signed-off-by: Jacob Keller [email protected]

This attempts to fix the BetterJunimosForestry mod to work with Stardew
Valley 1.6, though it is currently build-tested only.

The big changes include:

* switching to string IDs instead of numerical indexes
* fixing many function references
* re-wrote the fruit tree harvest logic entirely
* removed a bunch of references to debris logic that appears to have
  been removed from Stardew Valley 1.6
* Update the target to net6.0, and update the ModBuildConfig and JSON
  references.
* Various other fixes require to get things compiling
* Fix mod content loading to load icons and scroll2.png

Signed-off-by: Jacob Keller <[email protected]>
@jacob-keller jacob-keller force-pushed the jk-unofficial-better-junimos-forestry-sdv-1.6 branch from 5104fff to 8e44443 Compare April 10, 2024 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant