Fury pvp gs macro 6.2.3

would love feedback and maybe some help from other macro modders to help perfect or make this an even better macro thanks in advance

Talent’s: 2,1,2,1,1,3,3
Glyph’s: Major’s: Bull Rush, Rude Interruption, Blood Thirst
Minor’s: User Pref

Sequences["FuryPvP"] = {
StepFunction = [[
stepa = "123456"

limit = string.len(stepa) or 1
if stepc == nil then
stepc = 1
end
if stepc >= limit then
stepc = 1
else
stepc = stepc + 1
end
step = tonumber(strsub (stepa, stepc, stepc))
]],
PreMacro = [[
/targetenemy [noharm][dead]
]],

-- Macro 1
[[
/run("Executing macro 1! (Bloodthirst)")
/cast [nomod, nostance:1] Battle Stance
/cast [nomod] Charge
/castsequence [nomod] reset=nocombat Bloodthirst
/startattack
]],

-- Macro 2
[[
/run("Executing macro 2! (Raging Blow)")
/cast [nomod, nostance:1] Battle Stance
/castsequence [nomod, combat] Raging Blow
/cast [nomod] Charge
/startattack
]],

-- Macro 3
[[
/run("Executing macro 3! (Wild Strike)")
/cast [nomod, nostance:1] Battle Stance
/castsequence [nomod, combat] Wild Strike
/cast [nomod] Charge
/startattack 
]],

-- Macro 4
[[
/run("Executing macro 4! (Execute)")
/cast [nomod, nostance:1] Battle Stance
/castsequence [nomod, combat] !Execute
/cast [nomod] Charge
/cast [nomod, combat] Recklessness
/startattack
]],

-- Macro 5
[[
/run("Executing macro 5! (Storm Bolt)")
/cast [nomod, nostance:1] Battle Stance
/castsequence [nomod, combat] Storm Bolt
/cast [nomod] Charge
/cast [nomod, combat] Recklessness
/startattack
]],

-- Macro 6
[[
/run("Executing macro 6! (Siegebreaker)")
/cast [nomod, nostance:1] Siegebreaker
/castsequence [nomod, combat] Siegebreaker
/cast [nomod] Charge
/startattack
]],

PostMacro = [[
/startattack
/cast [combat] Recklessness
/cast [combat] Victory Rush
/run UIErrorsFrame:Clear()
]],
}

What Glyphs you are using?

sorry for the late response ill update the main post.