traffic light- please tell me of any problems

int Green = 11;
int Yellow = 12;
int Red = 13;
int RedPeople = 10;
int Blue = 7;
void setup() {
pinMode(Green, OUTPUT);
pinMode(Yellow, OUTPUT);
pinMode(Red, OUTPUT);
pinMode(RedPeople, OUTPUT);
pinMode(Blue, OUTPUT);

//green light on
digitalWrite(Green, HIGH);
digitalWrite(RedPeople, HIGH);
delay(1000);
digitalWrite(Green, LOW);

digitalWrite(Yellow, HIGH);
digitalWrite(RedPeople, HIGH);
delay(1000);
digitalWrite(Yellow, LOW);
digitalWrite(RedPeople, LOW);

//red light on
digitalWrite(Red, HIGH);
digitalWrite(Blue, HIGH);
delay(1000);
digitalWrite(Blue, LOW);

for(int= i; i<5;i++);
digitalWrite(RedPeople, HIGH);
delay(200);
digitalWrite(RedPeople, LOW);
delay(200);
}

// put your main code here, to run repeatedly:

If it works properly, no problem.

Please use code tags to post code.

int Green = 11;
 int Yellow = 12;
 int Red = 13;
 int RedPeople = 10;
 int Blue = 7;

for(int= i; i<5;i++);

Why use 2 bytes when 1 will do?
Anything less than 255 should be type byte vs int to save on SRAM memory.
You'll appreciate it in a big program later.

What is this?

for(int= i; i<5;i++);

Perhaps something like this was intended?

for(byte i=0; i<5;i++)  {

... but then a matching curly bracket "}" is needed somewhere.

And at some point you're going to have to put a loop() in even if it's just an empty one. Just the comment saying where it should be isn't enough.

Steve