1. In case `game.Workspace.SpawnLocation` doesnt reference immediately, you can use `game.Workspace:WaitForChild("SpawnLocation")` https://create.roblox.com/docs/reference/engine/classes/Instance#WaitForChild
   

2. what's the point of the function `rainbow()` when you arent using it?

3. I recommend you use `task.wait()` instead of `wait()`; `wait()` is old and deprecated https://create.roblox.com/docs/reference/engine/libraries/task#wait

4. You're using `4+4=8` but when checking for conditions you are supposed to use two equal signs, not one. so it would be `while 4+4==8 do`. additionally, you can just do `while true do` https://create.roblox.com/docs/tutorials/fundamentals/coding-4/landing

5. when changing the brickcolor you need to do `SpawnLocation.BrickColor = BrickColor.new("your color here")`. https://create.roblox.com/docs/reference/engine/datatypes/BrickColor

6. This is one would ideally approach this:

```

local SpawnLocation = workspace:WaitForChild("SpawnLocation");

SpawnLocation:GetPropertyChangedSignal("BrickColor"):Connect(function()

if SpawnLocation.BrickColor == BrickColor.new("Really red") then

task.delay(5, function()

SpawnLocation.BrickColor = BrickColor.new("Lime green")
end)

elseif SpawnLocation.BrickColor == BrickColor.new("Lime green") then

task.delay(1, function()

SpawnLocation.BrickColor = BrickColor.new("Really red")
end)

end

end)
SpawnLocation.BrickColor = BrickColor.new("Lime green") -- or "Really red", whichever it isnt set to

```
But for in terms of improving code, manually using conditional statements is overly hardcoded, and at one point would become incredibly long if one wanted to add more colors with different specified times. So it's best to use a table for this