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

Reorganization #44

Merged
merged 2 commits into from
Dec 19, 2021
Merged

Reorganization #44

merged 2 commits into from
Dec 19, 2021

Conversation

mitchej123
Copy link
Contributor

  • Move config and log to Hodgepodge
  • Rework Mixin List largely inspired by SinTh0r4s's mixin layout
  • Bump dep to spongemixins 1.4.0 so we can look for dev jars in the classpath
  • Organize mixin directories by mod
  • Add a configurable list of TargetedMods that are hard required, and should crash without, as opposed to silently failing the mixin and carrying on. Defaults to everything!
  • CompileOnly --> Compile Deps, so everything gets pulled in for runClient

* Move config and log to Hodgepodge
* Rework Mixin List largely inspired by SinTh0r4s's mixin layout
* Bump dep to spongemixins 1.4.0 so we can look for dev jars in the classpath
* Organize mixin directories by mod
* Add a configurable list of TargetedMods that are hard required, and should crash without, as opposed to silently failing the mixin and carrying on.  Defaults to everything!
@mitchej123 mitchej123 requested a review from a team December 19, 2021 07:28
build.gradle Show resolved Hide resolved
Copy link
Member

@SinTh0r4s SinTh0r4s left a comment

Choose a reason for hiding this comment

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

Checked only the build script and left some pointers

dependencies.gradle Show resolved Hide resolved
dependencies.gradle Outdated Show resolved Hide resolved
dependencies.gradle Outdated Show resolved Hide resolved
dependencies.gradle Outdated Show resolved Hide resolved
dependencies.gradle Outdated Show resolved Hide resolved
build.gradle Show resolved Hide resolved
@mitchej123 mitchej123 merged commit 7270eb4 into master Dec 19, 2021
@mitchej123 mitchej123 deleted the reorg branch December 19, 2021 23:53
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.

4 participants