neoforged / Bus

Event firing and listening framework, based on the event bus concept
GNU Lesser General Public License v2.1
3 stars 8 forks source link

Terminally deprecate generic events and @HasResult #19

Closed Technici4n closed 11 months ago

Technici4n commented 11 months ago

I only deprecated @HasResult (but not getResult, hasResult, etc...) to make event authors move away from the system without affecting event users.

Shadows-of-Fire commented 11 months ago

You may as well deprecate Result, getResult, and setResult in this PR.

Technici4n commented 11 months ago

You may as well deprecate Result, getResult, and setResult in this PR.

I don't want to deprecate them as long as we have events with a result in NeoForge, otherwise users will have to deal with the annoying warnings.