I think it makes sense if they choose not to enforce their ToS for a repeating macro for this, but it simaltaneously clearly is a violation of their ToS.
- Cheating: Create, use, offer, promote, advertise, make available and/or distribute the following or assist therein:
- cheats; i.e. methods not expressly authorized by Blizzard (whether accomplished using hardware, software, a combination thereof, or otherwise), influencing and/or facilitating gameplay, including exploits of any in-game bugs, and thereby granting you and/or any other user an advantage over other players not using such methods;
- bots; i.e. any code and/or software, not expressly authorized by Blizzard, that allows the automated control of a Game, or any other feature of the Platform, e.g. the automated control of a character in a Game;
- hacks; i.e. accessing or modifying the software of the Platform in any manner not expressly authorized by Blizzard; and/or
- any code and/or software, not expressly authorized by Blizzard, that can be used in connection with the Platform and/or any component or feature thereof which changes and/or facilitates the gameplay or other functionality;
They could also solve it by simply adding an option that makes evade hold to cast so nobody has to find solutions elsewhere.