search

minetest-mods / player_monoids

Monoidally-managed Minetest state modifications
Other
12 stars 8 forks source link

README.md: Return from example fold methods #8

Closed celeron55 closed 1 month ago

celeron55 commented 1 month ago

I was reading the README and got annoyed by this detail... so I decided to try out Github's in-browser editor and do a PR directly in the web browser.

I'm pretty sure I didn't get the indentation right because Github made it impossible to figure out what was going on in regards to that, but you get the idea. So, please fix the indentation that I likely messed up before merging this.

This fix is otherwise correct, right? I can't imagine how a fold method can work without returning anything and without saving any state either.

SmallJoker commented 1 month ago

You're right. https://github.com/minetest-mods/player_monoids/blob/08bc018f92b54732ade8c5e1ff38c404706e411e/standard_monoids.lua#L27-L33

I'll squash&merge this PR in a few days unless there are objections (just in case).