Skip to content

Conversation

@margalva
Copy link
Collaborator

The error message uses str(e), so this needs to be defined.

@margalva margalva assigned margalva and randallfrank and unassigned margalva Oct 16, 2023
@margalva margalva enabled auto-merge (squash) October 16, 2023 19:28
@codecov-commenter
Copy link

Codecov Report

Merging #72 (8183b80) into main (343bbf4) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   76.55%   76.55%           
=======================================
  Files          16       16           
  Lines        4845     4845           
  Branches     1026     1026           
=======================================
  Hits         3709     3709           
  Misses        796      796           
  Partials      340      340           
Files Coverage Δ
.../dynamicreporting/core/utils/geofile_processing.py 82.52% <0.00%> (ø)

@margalva margalva merged commit 4ba5fb7 into main Oct 16, 2023
@margalva margalva deleted the fix/except_syntax branch October 16, 2023 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants