Non-lowercase lookup_table_name breaks PR checks with Synthea
jrlegrand opened this issue · comments
Joseph LeGrand commented
Problem Statement
We camel_Cased our lookup_table_names originally, but lower_cased our .csv file names. This didn't break anything when running Synthea with MDT, but it breaks an automated GitHub PR check in the Synthea repo which will cause frustration moving forward.
Criteria for Success
MDT continues to run bug free and are still able to integrate with Synthea locally. No PR / merge errors when trying to submit future PRs to Synthea repo.
Additional Information
I aligned the way we lowercase the lookup_table_name with the way we lowercase the .csv file name by using the normalize_name function that we had created.