Léo Lam
Léo Lam
FYI, this was broken to fix #13.
Some features _are_ merged back to upstream; however it takes a lot of work (or at least it used to) since the code style is not exactly the same, and...
Only skimmed through this as this is a rather large PR and I don't really have time to give this a proper review today, but can you add line breaks...
So sorry for the delay - I'll take a look at the latest changes this weekend (and hopefully this can be merged then)
It seems the linter is unhappy about some of the formatting - could you run clang-format on the files it's complaining about? lgtm other than that
My two cents: just because the function is only used for fairies doesn't mean it should be called removeFairy, especially when it's generic (takes an item name + checks if...
Yeah, looks like that could be a regression... I'll investigate.