What is wrong with this script?

while true do
wait(2)
for i = 1,2 then
script.Parent.Position = Vector3.new(0, i, 0)
wait()
end

--You have 
for i = 1,2 then 

--it's actually 
for i = 1,2 do

You also need to put another end I believe.