Yes, your macros work in the sense they are functional. Is it optimal? Is it competitive? Absolutely not. You are essentially fighting a losing battle against a whole thread that is telling you why it is wrong with examples where not one person has agreed with you that it works in a competitive playing field. Maybe if you did your own research you'd find youtube videos where this has been tested somewhat extensively throughout the entirety of 2.0 and it all leads to the same conclusion - you do not macro your rotation. At the end of the day if this is how you want to play, that is fine but two things:
1. Don't claim you are "teaching" people through macros where a whole thread is telling you why this is wrong to do so.
2. Expect invites being thrown at you to do serious content when there is evidence out there that both mechanically and parse wise you are losing DPS.

To be honest I think this is a thinly veiled "My $15. My play style. Parsers are bad." thread and you managed to get a lot of bites. 10/10 though good job.