Skip to content

Conversation

@jtrobles-cdd
Copy link
Member

Fix the following warning:

cl_sii/cte/f29/parse_datos_obj.py:31:
ResourceWarning: unclosed file
<_io.TextIOWrapper name='cl_sii/data/cte/f29_datos_obj_missing_key_fixes.json' mode='r' encoding='UTF-8'>
CTE_F29_DATOS_OBJ_MISSING_KEY_FIXES: SiiCteF29DatosObjType = json.load(

Fix the following warning:

> cl_sii/cte/f29/parse_datos_obj.py:31:
> ResourceWarning: unclosed file
> <_io.TextIOWrapper name='cl_sii/data/cte/f29_datos_obj_missing_key_fixes.json' mode='r' encoding='UTF-8'>
> CTE_F29_DATOS_OBJ_MISSING_KEY_FIXES: SiiCteF29DatosObjType = json.load(
@jtrobles-cdd jtrobles-cdd added bug Something isn't working component: cte labels Sep 24, 2024
@jtrobles-cdd jtrobles-cdd self-assigned this Sep 24, 2024
@jtrobles-cdd jtrobles-cdd requested a review from a team as a code owner September 24, 2024 19:28
@sonarqubecloud
Copy link

@jtrobles-cdd jtrobles-cdd merged commit b886d6a into develop Sep 24, 2024
@jtrobles-cdd jtrobles-cdd deleted the bugfix/cte-python-warning branch September 24, 2024 19:48
@codecov
Copy link

codecov bot commented Sep 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.45%. Comparing base (665346c) to head (e6a312f).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #697   +/-   ##
========================================
  Coverage    87.45%   87.45%           
========================================
  Files           38       38           
  Lines         3092     3093    +1     
  Branches       381      382    +1     
========================================
+ Hits          2704     2705    +1     
  Misses         246      246           
  Partials       142      142           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@svillegas-cdd svillegas-cdd mentioned this pull request Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working component: cte

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants