Use None instead of NoReturn

[ian-r-rose]

Fixes #112

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (3103262) 100.00% compared to head (40f2466) 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #113   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           47        47           
  Lines         2194      2194           
  Branches       272       272           
=========================================
  Hits          2194      2194           

Impacted Files	Coverage Δ
dbt_checkpoint/generate_model_properties_file.py	100.00% <100.00%> (ø)
dbt_checkpoint/tracking.py	100.00% <100.00%> (ø)
dbt_checkpoint/unify_column_description.py	100.00% <100.00%> (ø)
dbt_checkpoint/utils.py	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[JFrackson]

This looks good to me! From looking into it, this is definitely a mistake on the implementation and this corrects it to use the right type. This is explained well here for anyone else who is curious.

[ian-r-rose]

Thanks for the review and merge!