fix(plugins): remove EcmwfSearch default dates#2008
Merged
Conversation
Contributor
Test Results 4 files ± 0 4 suites ±0 3m 40s ⏱️ +9s Results for commit 9863a15. ± Comparison against base commit 379db43. This pull request removes 1 and adds 5 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
Collaborator
Code Coverage (Ubuntu)DetailsDiff against developResults for commit: 9863a15 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
608a9f8 to
0b9d288
Compare
Collaborator
Code Coverage (Windows)DetailsDiff against developResults for commit: 9863a15 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
a298633 to
e7869a3
Compare
0c2fe9a to
c4d0bd5
Compare
fa7780c to
be21f29
Compare
sbrunato
requested changes
Feb 16, 2026
be21f29 to
d0553d7
Compare
d0553d7 to
4d13184
Compare
4d13184 to
5c8bec6
Compare
5c8bec6 to
f58a0e9
Compare
f58a0e9 to
9863a15
Compare
sbrunato
reviewed
Apr 8, 2026
Collaborator
sbrunato
left a comment
There was a problem hiding this comment.
I made some refactoring, moved dates parsing methods to eodag/utils/dates.py, used calendar from standard lib, and added doc for new functions.
@pdavid-cssopra if that looks ok for you, we can go to merge
sbrunato
approved these changes
Apr 8, 2026
anesson-cs
pushed a commit
that referenced
this pull request
Apr 17, 2026
Co-authored-by: Sylvain Brunato <sylvain.brunato@c-s.fr>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1904
Following #1877 where default values were removed for
EcmwfSearchproviders,dates_requiredplugin config parameterNote: there might be a silent issue while parsing some bad-formatted milliseconds in dates with dateutil.parser.isoparse