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

Feature: Kill burning entities when attempting to extinguish them with oil or fuel #165

Merged
merged 11 commits into from
Oct 1, 2021

Conversation

g307m
Copy link
Contributor

@g307m g307m commented Sep 30, 2021

Unlike last time, this is not an accidental PR :)

@JustPyrrha
Copy link
Contributor

is this ready for review?

Co-authored-by: Pyrrha <me@joezwet.me>
Copy link
Member

@marcus8448 marcus8448 left a comment

Choose a reason for hiding this comment

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

I'd prefer to use an @Accessor interface mixin than an access widener, as its much easier to maintain across mappings, refactors and minecraft versions.

@g307m
Copy link
Contributor Author

g307m commented Oct 1, 2021

This is my first 'real' contribution to something, I'd say I am definitely learning here. Got IDEA fired up and I'm going to work at this.

grantlmul added 2 commits September 30, 2021 21:07
# Conflicts:
#	src/main/java/dev/galacticraft/mod/mixin/EntityMixin.java
@g307m
Copy link
Contributor Author

g307m commented Oct 1, 2021

alright i think this is good now

@JustPyrrha JustPyrrha self-requested a review October 1, 2021 02:14
Copy link
Member

@marcus8448 marcus8448 left a comment

Choose a reason for hiding this comment

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

Could you revert your indentation changes to the access widener?

@g307m
Copy link
Contributor Author

g307m commented Oct 1, 2021

Indentation changes? I don't think I changed anything, I'll poke at that.

no more indentation! no more spaces!
@g307m
Copy link
Contributor Author

g307m commented Oct 1, 2021

Excessive indentation has been removed.

@g307m g307m requested a review from marcus8448 October 1, 2021 02:58
Copy link
Member

@marcus8448 marcus8448 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@g307m
Copy link
Contributor Author

g307m commented Oct 1, 2021

image

@craftablescience craftablescience merged commit 2ad1c81 into TeamGalacticraft:main Oct 1, 2021
marcus8448 added a commit that referenced this pull request Oct 19, 2021
* build: enable publishing to snapshot repo and publish snapshots from main pushes (#155)

* feat: prevent players from extinguishing themselves with flammable fluids

* chore: defer registration of objects to mod init (#156)

prevents some annoying classloading order bug

* fix: don't bother setting the executable bit in actions (#157)

its already set in the repo

* move damage source for oil boom to the proper location

* fix: register chunk generators (#160)

* cleanup: rei (#159)

* cleanup: rei
moves some common things out of the client package
also fixes a bug where circuit fabricators were treated like a 3x3 grid internally
uses default transfer handler that actually checks for materials

* chore: transfer handlers for the compressor and circuit fabricator

* chore: apply suggestions from code review

Co-authored-by: Pyrrha <me@joezwet.me>

* Feat: cave veins render correctly now (#163)

* fix: use correct texture for arc furnace face (#164)

* Slight change to make a difference

* Make texture work

* Apply 2/3 suggestions from code review

Co-authored-by: Pyrrha <me@joezwet.me>

* making changes for #165

* undo indentation changes

no more indentation! no more spaces!

* Feat: Kill burning entities when attempting to extinguish them with oil or fuel (#165)

* feat: prevent players from extinguishing themselves with flammable fluids

* move damage source for oil boom to the proper location

* Apply 2/3 suggestions from code review

Co-authored-by: Pyrrha <me@joezwet.me>

* making changes for #165

* undo indentation changes

no more indentation! no more spaces!

Co-authored-by: Pyrrha <me@joezwet.me>

* finish merging main

* rename launch_pad to rocket_launch_pad to both fix models and make things more 'glancable'

* fix: thermal leggings pattern + recipes (#166)

* Crafting for Nuggets

* Fixed wrong pattern

* Advanced panel recipe

* Dark Decoration Craft

Using Desh for the craft

* Applying Changes 1

* Applying Changes 2

Changed Recipe for it. With Craftablescience

* marcus: defer some registration

* Update src/main/java/dev/galacticraft/mod/block/entity/GalacticraftBlockEntityType.java

Co-authored-by: marcus8448 <28928887+marcus8448@users.noreply.github.com>

Co-authored-by: Pyrrha <me@joezwet.me>
Co-authored-by: marcus8448 <28928887+marcus8448@users.noreply.github.com>
Co-authored-by: mMONTAGEe <davidmamik10@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants