Skip to content

Conversation

@jtrh
Copy link
Contributor

@jtrh jtrh commented Jan 6, 2021

The zone attribute is not defined in the abstract base class
datetime.tzinfo. We need to check that it is there before using it in
the function validate_dt_tz to prevent unexpected exceptions when
dealing with Python Standard Library time zones that are instances of
class datetime.timezone.

The `zone` attribute is not defined in the abstract base class
`datetime.tzinfo`. We need to check that it is there before using it in
the function `validate_dt_tz` to prevent unexpected exceptions when
dealing with Python Standard Library time zones that are instances of
class `datetime.timezone`.
@jtrh jtrh self-assigned this Jan 6, 2021
@jtrh jtrh marked this pull request as ready for review January 6, 2021 20:36
@jtrh jtrh requested a review from glarrain January 6, 2021 20:36
@codecov-io
Copy link

codecov-io commented Jan 6, 2021

Codecov Report

Merging #175 (fe58daa) into develop (1f4ec01) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #175      +/-   ##
===========================================
+ Coverage    76.77%   76.79%   +0.02%     
===========================================
  Files           27       27              
  Lines         1834     1836       +2     
  Branches       308      308              
===========================================
+ Hits          1408     1410       +2     
  Misses         282      282              
  Partials       144      144              
Impacted Files Coverage Δ
cl_sii/libs/tz_utils.py 52.94% <100.00%> (+2.94%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f4ec01...fe58daa. Read the comment docs.

@jtrh jtrh merged commit 8d1e0d3 into develop Jan 6, 2021
@jtrh jtrh deleted the feature/add-checks-to-tz-utils-validate-dt-tz branch January 6, 2021 20:41
@jtrh jtrh mentioned this pull request Jan 11, 2021
@jtrobles-cdd jtrobles-cdd added enhancement New feature or request and removed feature labels Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: libs enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants