Cleanup on primary key position - Githubissues

smart-on-fhir / cumulus-library

https://docs.smarthealthit.org/cumulus/library/

Apache License 2.0

2 stars 0 forks source link

Cleanup on primary key position #172

Closed dogversioning closed 5 months ago

dogversioning commented 5 months ago

This PR makes the following changes:

moves primary key to first position in tables, renames it so that it is always id
removes test infrastructure to work around random primary key placement
adds a few missing derived tables to core unit testing.

Checklist

[X] Consider if documentation (like in docs/) needs to be updated
[X] Consider if tests should be added
[X] Run pylint if you're making changes beyond adding studies
[X] Update template repo if there are changes to study configuration