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

Breaking up flowers_plus mod #5

Merged
merged 1 commit into from Dec 14, 2021
Merged

Breaking up flowers_plus mod #5

merged 1 commit into from Dec 14, 2021

Conversation

JordanL2
Copy link
Contributor

@JordanL2 JordanL2 commented Dec 9, 2021

Splitting up flowers_plus into separate mods for seaweed, sunflowers and waterlilies, for clarity and so the user can disable them individually.

Fixed sunflower maximum heat.

Removed spawning, as we're already creating these things at worldgen.

@JordanL2
Copy link
Contributor Author

JordanL2 commented Dec 9, 2021

Also, there were some aliases defined at the bottom of the original init.lua, for compatibility when the farming mod came out and deprecated some things that were in this mod. These aliases were 4-8 years old, so I think it's safe to remove them now.

@wsor4035
Copy link
Contributor

wsor4035 commented Dec 9, 2021

if your going to create new mods, it is probably best to namespace them, especially considering how generic that names are. for example pl_seaweed, pl_waterlily, pl_sunflowers

@wsor4035
Copy link
Contributor

wsor4035 commented Dec 9, 2021

at least from https://content.minetest.net/metapackages/flowers_plus/ it seems the depreciating the flowers_plus modname is ok

@JordanL2
Copy link
Contributor Author

JordanL2 commented Dec 9, 2021

Good point, will rename them.

@JordanL2
Copy link
Contributor Author

Done, prefixed with "plantlife_" as that seems to be the convention with other modpacks.

@JordanL2
Copy link
Contributor Author

Added settings for each mod to control spawning.

@JordanL2
Copy link
Contributor Author

Since the along_shore mod does nothing, since it was all moved into flowers_plus a long time ago, I've removed it.

Also fixed plantlife_seaweed since it was generating "along_shore:seaweed" items, and then aliasing them to the "flowers:seaweed" items.

@wsor4035
Copy link
Contributor

Done, prefixed with "plantlife_" as that seems to be the convention with other modpacks.

might be a bit nit picky, but i would suggest a two letter abbreviation as it prevent you from having massive modnames. see nodecore, farlands_reloaded, mineclon*, etc.

@JordanL2
Copy link
Contributor Author

The longest is plantlife_waterlilies which is 21, you sure that's too long?

@JordanL2
Copy link
Contributor Author

I think it makes it much clearer what actual modpack it's from.

@JordanL2
Copy link
Contributor Author

Eh ok I'll rename them.

Splitting up flowers_plus into separate mods for seaweed, sunflowers and waterlillies, for clarity and so the user can disable them individually.

Fixed sunflower maximum heat.

Removed spawning, as we're already creating these things at worldgen.

Added settings for rarity and max count to each mod - waterlilies, seaweed and sunflowers

Removing along_shore mod as it does nothing now.
@JordanL2
Copy link
Contributor Author

Done.

@JordanL2
Copy link
Contributor Author

Sorry to bump but does this require anyone else's approval?

@BuckarooBanzay
Copy link
Member

Sorry to bump but does this require anyone else's approval?

Don't think so (i don't use that mod anywhere but the changes look good 👍)

@BuckarooBanzay BuckarooBanzay merged commit 1a96ac8 into mt-mods:master Dec 14, 2021
@JordanL2 JordanL2 deleted the disable_sunflowers branch December 16, 2021 09:41
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.

None yet

3 participants