Can't figure out why PCB is not working
What you can do to improve your design is scrap the pull-up resistors from the PCB and use the internal 20k pull-ups of the MCU instead. Then you can get rid of both the 5V trace and the external resistor footprints, reducing possible errors. You need to configure D2..D9 as INPUT_PULLUP instead of INPUT in your setup function, like this:

pinMode (2, INPUT_PULLUP);


As to why it doesn't work (apart from the obvious, that has already been pointed to you by others: when you close the "switch" you are pulling it LOW, not HIGH) it's impossible to tell without looking at your code. Once you simplify you circuit getting rid of the external pull-ups, if you keep not observing a level transition when "operating the switch" then it'll mean that something is failing in the code of your setup and/or loop functions.