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

Update Buildscript #3

Merged
merged 5 commits into from
Jan 29, 2022
Merged

Update Buildscript #3

merged 5 commits into from
Jan 29, 2022

Conversation

glowredman
Copy link
Member

@glowredman glowredman commented Jan 24, 2022

GTNewHorizons/ExampleMod1.7.10#12 needs to be reviewed/merged first!

@glowredman glowredman marked this pull request as ready for review January 28, 2022 21:35
@glowredman glowredman requested a review from a team January 28, 2022 21:35
@Dream-Master Dream-Master requested review from mitchej123, YannickMG and a team January 28, 2022 22:00
@YannickMG
Copy link

Buildscript builds, runClient runs, full modpack loads and runs. The cleanup done looks good but I'm no authority on whether it is up to the very latest best practices. I assume this would benefit from wide playtesting since this mod impacts a lot of recipes.

Copy link

@Glease Glease left a comment

Choose a reason for hiding this comment

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

Breaks inscriber compat. Otherwise is probably ok.

Copy link

@Glease Glease left a comment

Choose a reason for hiding this comment

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

Upon further examination this list appears to not actually be used. This is ok then I guess.

@Dream-Master Dream-Master merged commit 8989fdb into master Jan 29, 2022
@Dream-Master Dream-Master deleted the update-buildscript branch January 29, 2022 07:57
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