Currently the attack event on the Melee class is not really useful since it only have it's handler passed in the callback

Having the "Attack" event closer to the Damageable "TakeDamage" event would make this event more meaningful (currently I see no reason to use it over TakeDamage)