Skip to content

Remove legacy shaded MixinExtra and GasStation module#61

Merged
ah-OOG-ah merged 10 commits intomasterfrom
maint
Dec 25, 2025
Merged

Remove legacy shaded MixinExtra and GasStation module#61
ah-OOG-ah merged 10 commits intomasterfrom
maint

Conversation

@ah-OOG-ah
Copy link
Copy Markdown
Member

@ah-OOG-ah ah-OOG-ah commented Oct 26, 2025

Searching on GitHub brings up no current mods using this, and it's been about two years since it was last possible to reference from an IDE (see 87e71b1).

Also:

  • Removes the previously-deprecated GasStation module. Note: this requires a major version bump to 0.2.0.
  • Removes the GasMix mixin flavor. See above.

@ah-OOG-ah ah-OOG-ah marked this pull request as draft October 26, 2025 17:43
@ah-OOG-ah ah-OOG-ah changed the title Remove legacy shaded MixinExtra from GTNHMixins module Remove legacy shaded MixinExtra and GasStation module Dec 16, 2025
@ah-OOG-ah ah-OOG-ah marked this pull request as ready for review December 17, 2025 20:40
@ah-OOG-ah
Copy link
Copy Markdown
Member Author

This crashes in NH daily 243, due to MixinExtra not loading for a preinit mixin. Unsure if this broke MixinExtra in general, or just in preinit

@ah-OOG-ah
Copy link
Copy Markdown
Member Author

Update: fixed the issue, this should be ready for testing now

@ah-OOG-ah
Copy link
Copy Markdown
Member Author

Tested it in GTNH (daily 243) and one version of @brandyyn's modpack, no breakage found.

@ah-OOG-ah ah-OOG-ah merged commit cb08fbd into master Dec 25, 2025
2 checks passed
Copy link
Copy Markdown
Contributor

@basdxz basdxz left a comment

Choose a reason for hiding this comment

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

Looks fine, outside of the unmentioned import and formatting changes which really should have been their own separate housekeeping pr.

@Roadhog360
Copy link
Copy Markdown
Member

Spotless forces that I think.

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.

5 participants