Richard Harrah

I concur with emallson's proposal. I also think we should recommend a consolidated spells file per spec with individual exports. Having all of the spells defined together is useful instead...

> If we actually need the ability to index by id, that is. I haven't done a pass on what the use case for that is right now, but it...

> If we actually need the ability to index by id, that is. Another use case is in `useSpellInfo`, which is used by all `SpellLink` components: ``` useEffect(() => {...

Setup instructions work for Windows.

This issue should be able to be closed; the linked report is showing as not cheap.

> This has a lot of copy-paste from Buffs, maybe this should be even more tightly integrated with the existing Buffs module - do we even need a separate analyzer?...

> > I don't disagree; especially since this original case is for debuffs that are actually buffs. I think we'd still need some way to denote weird cases like these...

I'm not sure how we would implement something of that nature, but we may in the future. Thanks for the suggestion.

Lightning seems to be bugged. It will be looked into.

That's not the full error. I need more information after the NullPointerException.