[Jarewa0606]

A po co blokujesz?? musi być być "while"?

if (timeForFeed == 1)
{
if (digitalRead(1) == HIGH)
{
pumpOn();
}else {
pumpOff();
vccOff();
}
timeForFeed = 0;
timeOne = millis();
timeTwo = timeOne;
}

choć timeForFeed jest bez sensu po co blokować odczyty na 5s??

uint8_t Odczyt=(digitalRead(1) );

if(Odczyt != ostatniOdczyt){
timeOne = millis();
}
ostatniOdczyt=Odczyt;

If(millis()-timeOne >=50) {
if(odczyt != stanOdczyt){
stanOdczyt=odczyt;
if(stanOdczyt=HIGH){
pumpOn();
}else{
pumpOff();
vccOff();
}
}
}