I feel stupid asking this but?

I'm very new to this, and don't really understand it all that well, but I'm giving it a go, what's wrong here?

while true do
rs.setOutput(right, true)
sleep(1)
rs.setOutput(right, false)
end

Cheers

Lowprofile

The side name needs to be a string, so use "right" instead of right.

Tip: If there is not quotes on rs.setOutput(right,true) it means that your describing a variable but if you put quotes on both sides it means that your describing a string or the side of your rs output is.

Heres the Logic of rs.setOutput(right, true):

local right = "right"
local left = "left"
rs.setOutput(right, true)
rs.setOutput(left, true)

Heres the Logic with quotes:

rs.setOutput("right", true)
rs.setOutput("left", true)

Also, you have no delay inbetween the last redstone command and first, this will cause the redstone to always be on.