Honestly, compared to people who doesn't even know how to do the proper rotations (either don't bother learning them, or can't put in enough time to practice them), this macro is probably a step up. I haven't personally tested this, but I imagine losing a GCD every 10 is not going to make him the worst player you will encounter in df. He is not going to be "good", but I'm sure it will be adequate enough for most (but not all) content.

The skill bloat is something that SE has acknowledged to be a problem that they will address in the future. If by using this, someone who doesn't have a lot of time or just want to play the game leisurely can improve their overall dps (because learning the proper rotation is easier when there are much fewer buttons to press), I don't see the harm. This does more or less precludes them from improving further if they want to start tackling more serious content in the future, unless they relearn everything from scratch again. In other words, this is not really a training wheel into the proper way of dpsing, so that's the downside. But if they are sure that they are never going to do those harder contents...why not?

Having said that, I do think the OP should put a large disclaimer that this is not the optimal way to play, in the first post and in his video.