int Button = 1; // button
Pin 0 and 1 are already used by the USB<>Serial chip, so don't use these.
const byte buttonPin = 2; // button pin 2
const byte = 3; // valve pin 3
etc.
Best to use constant byte for values < 255 that do not change, like pin numbers. It saves memory.
pinMode(Button, INPUT_PULLUP); // internal pull up resistor assumes you are going to connect the button between pin and ground.
The pin is HIGH (from the pull up) when the button is not pressed (open circuit).
while(digitalRead(Button) == 0); // wait until button is pushed released
This assumes to wait here while the button is held down. Did you want that?
while(digitalRead(buttonPin); // this is the same...
while(digitalRead(buttonPin == HIGH); //as this
It means: wait here while the buttonPin is HIGH (not pressed).
Remove the two brackets there.
Next time use code tags. Read the "how to post" sticky on top of every main page.
Leo..