Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use pendulum to parse dates #123

Merged

Conversation

renaudjester
Copy link
Collaborator

Fix CMT-100 and simplify the parsing of the dates

@renaudjester renaudjester force-pushed the copernicusmarine-toolbox-v2 branch from 4ef9bb9 to 84d2dd3 Compare August 29, 2024 10:37
@renaudjester renaudjester changed the base branch from copernicusmarine-toolbox-v2 to main August 29, 2024 10:46
@renaudjester renaudjester changed the base branch from main to copernicusmarine-toolbox-v2 August 29, 2024 10:46
@renaudjester renaudjester force-pushed the fix-v2-format-iso-retention-period branch from d489a54 to 49229bd Compare August 29, 2024 10:53
Not using any cache anymore
Always get the data on the fly

fix: use pendulum to parse dates

chore: use pendulum accross the repo

fix: to pendulum datetime from boto3

fix: microsecond problem with boto3
@renaudjester renaudjester force-pushed the fix-v2-format-iso-retention-period branch from 49229bd to 5c01ffd Compare August 29, 2024 12:39
@renaudjester renaudjester requested a review from uriii3 August 29, 2024 12:55
@renaudjester renaudjester merged commit b65547a into copernicusmarine-toolbox-v2 Sep 3, 2024
2 checks passed
@renaudjester renaudjester deleted the fix-v2-format-iso-retention-period branch September 3, 2024 12:58
renaudjester added a commit that referenced this pull request Oct 28, 2024
Support more date format: support timezones and uses pendulum
renaudjester added a commit that referenced this pull request Oct 28, 2024
Support more date format: support timezones and uses pendulum
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.

2 participants