Skip to content

Added Support for Enchanted - Witchcraft - #5

Closed
ArtpokeBr wants to merge 2 commits into
davenonymous:1.21.1from
ArtpokeBr:1.21.1
Closed

Added Support for Enchanted - Witchcraft#5
ArtpokeBr wants to merge 2 commits into
davenonymous:1.21.1from
ArtpokeBr:1.21.1

Conversation

@ArtpokeBr

Copy link
Copy Markdown
Contributor

I hope this is right, I tried to follow the guide :)

Comment thread mods.conf
smartbrainlib 661293 6668303 # 2025-06-18T21:21:13.86Z | SmartBrainLib-neoforge-1.21.1-1.16.8.jar
kotlin-for-forge 351264 6497906 # 2025-05-05T07:11:02.707Z | kotlinforforge-5.8.0-all.jar
roots 246183 6664961 # 2025-06-18T06:52:53.79Z | roots-4.0.0.7-alpha.jar
enchanted-witchcraft 560363 6668864

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

6668864 is the latest file for Fabric.
6668863 is the Neoforge version, which we need here.

And murphy obviously had to hit you with one of the corner cases. Enchanted Witchcraft has extra dependencies on Modopedia and StateObserver. All in all the PR has to look like this:

enchanted-witchcraft                     560363       6668863
modopedia                                1132038      6673794
stateobserver                            701213       6522771

@davenonymous

Copy link
Copy Markdown
Owner

I've applied these changes and also added Malum (and its dependencies).

You can find the packs in the generated folder or in this zip: for-artpoke.zip

The next version of Bonsai Trees will include these by default.

@ArtpokeBr

Copy link
Copy Markdown
Contributor Author

Nice! sorry for the inconvenience btw Dave, it seems that I was not very lucky with this ''-.-

@davenonymous

Copy link
Copy Markdown
Owner

Not an inconvenience at all. Quite the opposite. Good test that shows the system needs some more refinement :)

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.

2 participants