Results 57 comments of 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.