Minor nitpick about their exit code technique [0]: The command checks if the table exists, but it does not appear to re-run if the source file has been updated.

Minor nitpick about their exit code technique [0]: The command checks if the table exists, but it does not appear to re-run if the source file has been updated. Usually with Make you expect it to re-run the database load if the source file has changed. It's better to use empty targets [1] to track when the file has last been loaded and re-run if the dependency has been changed. [0] https://github.com/propublica/ilcampaigncash/blob/master/Makefile#L27 https://github.com/propublica/ilcampaigncash/blob/master/Mak... [1] https://www.gnu.org/software/make/manual/html_node/Empty-Targets.html https://www.gnu.org/software/make/manual/html_node/Empty-Tar...