Pin numbers don't match b/w datasheet and variant.h!

That is the variant file for the M0 Pro:

I'm guessing the reference to "Zero board" is because they copied it over from the Zero and didn't bother to completely update the comment text.

This is the variant for the Zero:

and indeed it does show the pins 2 and 4 switched from the arduino_mzero variant.