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

Infinite Energy Emitter (from CEU) #21

Merged
merged 4 commits into from
Jul 6, 2021

Conversation

bruberu
Copy link
Member

@bruberu bruberu commented Jul 5, 2021

What:
This particular PR addresses an issue in testing where it is often difficult to test certain machines and recipes without a singleblock energy source, which becomes an especially pronounced problem for the higher tiers.

How solved:
To address this, I have imported the Infinite GTEU Energy Emitter from CEU. For the most part, this implementation was copied entirely, although I did decide to do a little refactoring with the GUI builder, since I could not find an easy way to work around it. Later on, the code there should probably be cleaned up, although I do not have the skills to pull off such a thing. Furthermore, a rather large excess of extraneous classes, used for the rest of CEU's infinite energy handling tile entities, have also been imported, to serve both the purpose of leaving the door open for importing the rest in, and to serve the purpose of making this less of a headache for me to handle.

Outcome:
Adds a new MTE for infinite GTEU generation.

Possible compatibility issue:
None are expected.

@serenibyss
Copy link
Member

If you are using Intellij, can you please run the Reformat Code function on these files, excluding en_us.lang. You can find it by right-clicking on the files themselves on the left

@bruberu
Copy link
Member Author

bruberu commented Jul 5, 2021

Cleaned up code in the latest commit, excluding en_us.lang.

@bruberu
Copy link
Member Author

bruberu commented Jul 6, 2021

Fixed all merge conflicts.

@serenibyss serenibyss merged commit be14a10 into GregTechCEu:master Jul 6, 2021
@bruberu bruberu deleted the creativeEnergy branch July 6, 2021 03:34
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