We have a history, even recently, of not using PRIMARY KEYs and many of the cases seem tragically just because someone was unaware that it is good practice / maybe cribbed off earlier schema deltas. In many cases a UNIQUE index was created instead when that could've been a PRIMARY KEY.
This PR adds a test that flags up missing PRIMARY KEYs.
We have a history, even recently, of not using PRIMARY KEYs and many of the cases seem tragically just because someone was unaware that it is good practice / maybe cribbed off earlier schema deltas. In many cases a
UNIQUE
index was created instead when that could've been aPRIMARY KEY
.This PR adds a test that flags up missing PRIMARY KEYs.
Base:
develop
Original commit schedule, with full messages: