I also believe that round holes are just fine. Using “oval” holes can be a problem during production. Round holes are just drilled while “oval” holes need a router bit that goes sideways through the FR4.
A while ago I also made:
Showing the bottom (and thus mirrored) view in datasheets is a PITA. Good of eelik to catch that. In that mini tutorial I wrote in 2017 I also printed the layout in mirror image to have the PCB pads shown from the bottom of the part, and it’s not entirely certain if you would catch this mistake with that method. But anyway, it’s always a good idea to have the parts and do some kind of fitting before you send out Gerbers to a PCB Fab.