JerichoPenumbra
Explorer
Main reason that I don't quite like it as class feature is that many old items and feats that helped give bonuses or at least were useable with EB no longer function due to their wording specifying spells cast rather than spell attacks, as well as having a degree anti-synergy with a class like Marshal. While it should be easy enough to houserule common sense interpretations so that o5e stuff that worked with it before works again, or have EB be treated as "weapon" attack for the Ray, Scythe, and Whip or as a cantrip for Disturbance for classes that give other classes actions of some kind, I know that not everyone has a group that won't be stringent with rules wording. Or to put it plainly, try to have wording that allows for future compatibility (tough, I know) and maintains backwards compatibility (which is supposedly one of the selling points they try to do with a5e)Personally I prefer it this way, I really dislike 5.5e's approach of making everything a spell.
With the same rationale otherwise also the herald's smites would be just another spell, etc etc. Instead of having generic spells with ad hoc synergies with the class casting them (not good for future portability), I think it's better to roll them into core class features