Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

feat: added ice cream #54

Merged
merged 3 commits into from
Mar 11, 2024
Merged

feat: added ice cream #54

merged 3 commits into from
Mar 11, 2024

Conversation

wiggle1000
Copy link
Contributor

Adds a few flavors of ice cream, which is clearly a mandatory feature.

I have tried my best to keep it as clean and on-style as possible, but this is my first time working with Fabric and Kotlin, so I may have goofed up somewhere!

Copy link

@Lignum Lignum left a comment

Choose a reason for hiding this comment

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

Some minor syntactical concerns, most if not all of which should be fixed by the IntelliJ formatter. Not sure if this fits sc-goodies, but looks mergeable otherwise.

src/main/kotlin/io/sc3/goodies/Registration.kt Outdated Show resolved Hide resolved
src/main/kotlin/io/sc3/goodies/misc/IceCreamItem.kt Outdated Show resolved Hide resolved
src/main/kotlin/io/sc3/goodies/misc/IceCreamItem.kt Outdated Show resolved Hide resolved
src/main/kotlin/io/sc3/goodies/misc/IceCreamItem.kt Outdated Show resolved Hide resolved
src/main/kotlin/io/sc3/goodies/misc/IceCreamItem.kt Outdated Show resolved Hide resolved
src/main/kotlin/io/sc3/goodies/Registration.kt Outdated Show resolved Hide resolved
@wiggle1000
Copy link
Contributor Author

Updated with syntax fixes. Sorry for the bracket mess haha, the C++ muscle memory is coming for my life.

In defense of the off-topicness of the feature, I have to admit that this isn't really a direct replacement for anything we had on SC2. A lot of the community members recently (as I'm sure you've seen) have been increasingly interested in adding food items. I was thinking that this would be a fun, vanilla-style, and lightweight way to scratch a bit of that itch while people potentially implement more complex food systems.

Comment on lines +401 to +409
val iceCreamVanilla = rIceCreamItem("icecream_vanilla")
val iceCreamChocolate = rIceCreamItem("icecream_chocolate")
val iceCreamSweetBerry = rIceCreamItem("icecream_sweetberry")
val iceCreamNeapolitan = rIceCreamItem("icecream_neapolitan")
val iceCreamSpruce = rIceCreamItem("icecream_spruce")
val iceCreamMelon = rIceCreamItem("icecream_melon")
val iceCreamBeetroot = rIceCreamItem("icecream_beetroot")
// This is a bit messy, but a much simpler way to have the sundae be worth more food
val iceCreamSundae = rItem("icecream_sundae", IceCreamItem(itemSettings()
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: should there be an underscore between ice and cream because it's two words?

Suggested change
val iceCreamVanilla = rIceCreamItem("icecream_vanilla")
val iceCreamChocolate = rIceCreamItem("icecream_chocolate")
val iceCreamSweetBerry = rIceCreamItem("icecream_sweetberry")
val iceCreamNeapolitan = rIceCreamItem("icecream_neapolitan")
val iceCreamSpruce = rIceCreamItem("icecream_spruce")
val iceCreamMelon = rIceCreamItem("icecream_melon")
val iceCreamBeetroot = rIceCreamItem("icecream_beetroot")
// This is a bit messy, but a much simpler way to have the sundae be worth more food
val iceCreamSundae = rItem("icecream_sundae", IceCreamItem(itemSettings()
val iceCreamVanilla = rIceCreamItem("ice_cream_vanilla")
val iceCreamChocolate = rIceCreamItem("ice_cream_chocolate")
val iceCreamSweetBerry = rIceCreamItem("ice_cream_sweetberry")
val iceCreamNeapolitan = rIceCreamItem("ice_cream_neapolitan")
val iceCreamSpruce = rIceCreamItem("ice_cream_spruce")
val iceCreamMelon = rIceCreamItem("ice_cream_melon")
val iceCreamBeetroot = rIceCreamItem("ice_cream_beetroot")
// This is a bit messy, but a much simpler way to have the sundae be worth more food
val iceCreamSundae = rItem("ice_cream_sundae", IceCreamItem(itemSettings()

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh yeah, this is probably more proper! I didn't think about it that hard to be honest.

@Lemmmy
Copy link
Member

Lemmmy commented Jan 13, 2024

Thanks very much for taking the time to submit this PR; did you make the sprites yourself?

A lot of the community members recently (as I'm sure you've seen) have been increasingly interested in adding food items.

I sat on these PRs for a while because I believed at the time that it was just a fad. Out of the ones that came up, this one is the most likely to get merged, but I'm still not sure why it should be, or what value it would add.

I'm all down for a bit of silly fun, and accepting one or two food PRs might be relatively harmless, a bit of the unique SwitchCraft character. But, I don't really want to set a precedent here, potentially opening the floodgates to dozens of low-quality, single-file food PRs. At what point does it start becoming ridiculous? We already have one useless food item: popcorn… I'm aware that #4 is still sitting around.

@Lemmmy Lemmmy added the enhancement New feature or request label Jan 13, 2024
@wiggle1000
Copy link
Contributor Author

wiggle1000 commented Jan 30, 2024

Thanks very much for taking the time to submit this PR; did you make the sprites yourself?

I did! There shouldn't be any licensing issues.

But, I don't really want to set a precedent here, potentially opening the floodgates to dozens of low-quality, single-file food PRs. At what point does it start becoming ridiculous?

This is definitely an important consideration. I personally think it might be best to just judge it on a per-request basis. The community is pretty small, so I can't imagine it getting out of hand very easily.

We already have one useless food item

I disagree! Silly fun is the beating heart of Switchcraft!! Most things can be done with Computercraft and the 3d printers, but still not everything. If there were printer flags for more types of items (food, sit-on-able, etc), I think it would reduce the need for features like these, but that's a totally different conversation about a separate mod. Haha!

P.S. Sorry for the late reply!! I didn't get the notification for some reason

@Lemmmy Lemmmy merged commit ced16e3 into SwitchCraftCC:1.20.1 Mar 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants