@Mondetechno I see the schematic has some overlapping wires where I can't tell if the wires are connected or not. Maybe you know, but the CAD program does care, especially if you use the same program to draw the PCB, it will try to make you connect them when maybe they aren't supposed to be connected.

By the way, Arduino usually has internal pullup capability. Don't know which types of Arduino have it. If this one has it, then you don't actually need the resistors, because they are built into the Arduino

you set the pin mode to INPUT_PULLUP instead of just INPUT and then you get the internal pullup

But you can do it either way. Not every microcontroller has internal pullups, so maybe it makes sense to practice adding your own ones anyway

Yes, i use the Arduino Nano and it has internal pullup capability, so no resistor needed :)

Hi ... just seeing this is in chat now. @Mondetechno is there a reason you don't want to use the proper tools eg KiCAD? You will save yourself an enormous amount of time, even though it takes a little while to learn to use them. Many of your issues (overlapping lines, lines too close, tracks not matching circuit diagram) are checked or prevented automatically in proper CAD tools. Your board is nice and simple and a great one to start using such a tool.